Skip to content

Implement the create proxy request for map proxy #454

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Oct 3, 2018

Conversation

ihsandemir
Copy link
Collaborator

Implemented the create proxy request for map proxy.

fixes #54

@ihsandemir ihsandemir added this to the 3.10.2 milestone Sep 21, 2018
@ihsandemir ihsandemir self-assigned this Sep 21, 2018
@ihsandemir ihsandemir requested a review from sancar September 21, 2018 16:13
@devOpsHazelcast
Copy link
Contributor

Linux test FAILed.

@devOpsHazelcast
Copy link
Contributor

Windows test FAILed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@@ -146,6 +145,18 @@ namespace hazelcast {
DEFINE_EXCEPTION_CLASS(FutureUninitialized, protocol::FUTURE_UNINITIALIZED);
DEFINE_EXCEPTION_CLASS(ConsistencyLostException, protocol::CONSISTENCY_LOST);

class MemberLeftException : public ExecutionException {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This exception should be retryable.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, fixed.

…ble MemberLeftException be able to derive from both the ExecutionException and RetryableHazelcastException so that the d'amond problem is avoided.
@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@ihsandemir ihsandemir merged commit 5f62040 into hazelcast:master Oct 3, 2018
@ihsandemir ihsandemir deleted the createMapProxy branch October 3, 2018 09:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Distributed Object creation
3 participants