You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geode.apache.org by Darrel Schneider <ds...@pivotal.io> on 2015/08/13 00:11:12 UTC

Review Request 37421: GEODE-212: ResourceManager hurts off-heap performance

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37421/
-----------------------------------------------------------

Review request for geode and Kirk Lund.


Bugs: GEODE-212
    https://issues.apache.org/jira/browse/GEODE-212


Repository: geode


Description
-------

Added mightSendEvent method to optimize off-heap resource manager event delivery.
Also no longer call System.currentTimeMillis every time off-heap memory changes and we are evicting or critical.
Also got rid of two unneeded volatiles.


Diffs
-----

  gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/control/MemoryEvent.java bda6518df96912c4a3691d9beca24e9821c77c8f 
  gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/control/OffHeapMemoryMonitor.java a1856e4917c612b91c8db5e7628682efa796d2e9 

Diff: https://reviews.apache.org/r/37421/diff/


Testing
-------

precheckin


Thanks,

Darrel Schneider