-
Notifications
You must be signed in to change notification settings - Fork 36
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
Issue 44 - Support Hibernate 5.3.x #48
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've reviewed this before, and the rework from the original pull request looks good.
...t-hibernate53/src/main/java/com/hazelcast/hibernate/AbstractHazelcastCacheRegionFactory.java
Outdated
Show resolved
Hide resolved
); | ||
} | ||
|
||
protected abstract RegionCache createTimestampsRegionCache(String regionName, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More here. Perhaps one of the Hazelcast maintainers can link us to something re: their code style
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
hazelcast-hibernate53/src/main/java/com/hazelcast/hibernate/RegionCache.java
Show resolved
Hide resolved
hazelcast-hibernate53/src/main/java/com/hazelcast/hibernate/local/LocalRegionCache.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @dohertyfjatl for the PR. It's a piece of great work 👍
I'll merge it and we'll release it soon.
What I'm a little concerned is that for each version we create a new project and copy the source, which is hard to maintain. I think we should integrate this project back to hazelcast-hibernate
and use versions to distinguish the Hibernate versions. Anyway, it's a separate topic and we can do it later.
Thank you so much, we were waiting for it. |
FYI: hazelcast-hibernate53 (v1.3.0) is released. |
Awesome, thanks a lot!! I will test tomorrow |
Implements issue #44 by adding a new module, "hazelcast-hibernate53", which adds support for Hibernate 5.3.x: