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
TenantControl - creating cache records must use tenant #17121
TenantControl - creating cache records must use tenant #17121
Conversation
when factories get invoked, appropriate tenant environment is created
hazelcast/src/main/java/com/hazelcast/cache/impl/AbstractCacheRecordStore.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.
looks good, just one comment about exception throwing. Also, run mvn -Pcheckstyle clean install
locally to ensure you fixed the checkstyle errors. Here is the output from PR builder:
04:46:51 [INFO] --- maven-checkstyle-plugin:2.15:checkstyle (default) @ hazelcast ---
04:47:24 [INFO] Starting audit...
04:47:24 /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-builder/hazelcast/src/main/java/com/hazelcast/cache/impl/AbstractCacheRecordStore.java:153:5: Method length is 69 lines (max allowed is 60).
04:47:24 Audit done.
- moved some of constructor's functionality into a private method to comply with checkstyle rules
I have fixed the failed test |
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.
👍
Version 3.12.X
Wrapped factory instantiation in try/finally block with tenant control enabled.
This is so factories have their tenant environment set
Please view diff w/o whitespace changes to make it a very easy diff :)
4.0 version of this PR #17122