Skip to content

Instantly share code, notes, and snippets.

@galderz
Created October 23, 2014 15:56
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save galderz/62ae5120f5ac50ceabef to your computer and use it in GitHub Desktop.
Save galderz/62ae5120f5ac50ceabef to your computer and use it in GitHub Desktop.
[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