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
ISPN 5836 Managed eviction size #3744
ISPN 5836 Managed eviction size #3744
Conversation
The changes look OK to me but what about our plans for exposing the configuration over JMX dynamically (as mentioned in https://github.com/infinispan/infinispan/wiki/Dynamic-JMX-exposer-for-Configuration)? |
That's the plan for the future (maybe even within the 8.1 scope). But I wanted a solution now because of https://bugzilla.redhat.com/show_bug.cgi?id=1238206 |
Ok. In that case 👍 from me. |
- ISPN-5340 Add a Configuration MBean - ISPN-5836 Add JMX attribute and DMR handler for changing eviction size at runtime
141e496
to
787530e
Compare
I fixed the inconsistency: resize now throws the exception |
Pulling.. |
Integrated into master, thanks @tristantarrant ! |
long size = (long)(1.0 + (long)initialCapacity / loadFactor); | ||
int cap = (size >= (long)MAXIMUM_CAPACITY) ? | ||
long size = (long)(1.0 + initialCapacity / loadFactor); | ||
int cap = (size >= MAXIMUM_CAPACITY) ? |
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.
@wburns Do we even need do keep this constructor around?
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.
Probably not, tbh I just ignored the changes in this file since they seemed completely unnecessary but fine.
ISPN-5836 Expose eviction size modification through JMX and DMR
ISPN-5837 Deprecate eviction maxEntries configuration
ISPN-5340 Add a Configuration MBean
https://issues.jboss.org/browse/ISPN-5836
https://issues.jboss.org/browse/ISPN-5837
https://issues.jboss.org/browse/ISPN-5340