-
-
Save galderz/62ae5120f5ac50ceabef to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[17:28] <wburns> galderz, dberindei , pruivo : see this is where I think we have a disconnect, we talked about this before that JMX size probably doesn't need to be the most up to date value | |
[17:29] <galderz> dberindei, pruivo, wburns: that'd be fine by me too... it's just that QE has created a load of tests that rely on precise jmx size stat | |
[17:29] <pruivo> wburns: agree that it must not need to be the most up to date but it shouldnt count the expired entries :/ | |
[17:30] <vblagoje> wburns galderz I did not fix that. Someone else commited a fix, I think it was you Galder, no? | |
[17:30] <galderz> fix what? | |
[17:31] <wburns> galderz, pruivo: I can agree with that, however remote entries and loader shouldn't be counted it seems to me, no? | |
[17:33] <pruivo> wburns: agree | |
[17:34] <wburns> pruivo: although if we are removing expired entries it is really most up to date value btw :) | |
[17:34] <wburns> pruivo: that we can get at least | |
[17:35] <pruivo> wburns: you have a point there | |
[17:35] <pruivo> wburns: but the expired entries can take long time to be removed | |
[17:35] <wburns> pruivo: yeah :/ | |
[17:35] <pruivo> wburns: except if we remove them eagerly (when a get is performed) | |
[17:36] <pruivo> wburns: but... I'm not sure |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment