-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for Hazelcast v3.9 #19
base: master
Are you sure you want to change the base?
Conversation
Passing the ClientContext means it'll be a breaking change, isn't it? Yet ... another ... reflective ... factory ;) |
Btw thanks for looking into it already! :) Unfortunately I have some other stuff I work on right now but glad to see you're working on it! |
Well, it might be a breaking change for 3.7, but it works for 3.8 and 3.9 now. So I am not sure how to handle that and provide backward compatibility. |
Here it is, compatibility fixes for 3.8! |
I actually think to release V2 just now (with support for 3.7/3.8) and remove the old versions for V3 which would be 3.9+, what do you think? |
Yes, definitely. There are too many changes to maintain. |
is there any problem? |
This seems sufficient to support 3.9-SNAPSHOT as-is now. Does not solve lots of
@Deprecated
stuff.