----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58397/#review171896 -----------------------------------------------------------
Ship it! Ship It! - Eric Shu On April 12, 2017, 11:42 p.m., Darrel Schneider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58397/ > ----------------------------------------------------------- > > (Updated April 12, 2017, 11:42 p.m.) > > > Review request for geode, anilkumar gingade, Eric Shu, and Lynn Gallinat. > > > Bugs: GEODE-2097 > https://issues.apache.org/jira/browse/GEODE-2097 > > > Repository: geode > > > Description > ------- > > During disk recovery, the code now checks the offheap LRU limit instead of > the heap LRU limit for offheap regions. > A unit test has been added that without this fix would run out of offheap > memory during recovery. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/internal/cache/AbstractLRURegionMap.java > 328ff35b940239bfbcf817d144d97f886506b33a > > geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java > eaababaf6be6041c72156a338c69fa81f10db4c5 > geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java > d0aacc2d95fa38b1bc9faf68de795af5ef3d9090 > > geode-core/src/main/java/org/apache/geode/internal/cache/PlaceHolderDiskRegion.java > db01162aeeba8113c208565d683e71a41d0f6d00 > > geode-core/src/main/java/org/apache/geode/internal/cache/ProxyRegionMap.java > 92c7b6f8e90a3382e7ce00064581d69f9706fcd3 > > geode-core/src/main/java/org/apache/geode/internal/cache/control/HeapMemoryMonitor.java > afc9a23aafba2c1f993b30546f24cd2b5d6a0126 > > geode-core/src/main/java/org/apache/geode/internal/cache/control/InternalResourceManager.java > d16cd98bbb3eefc785a22f41441c1490060d0781 > > geode-core/src/main/java/org/apache/geode/internal/cache/control/MemoryMonitor.java > PRE-CREATION > > geode-core/src/main/java/org/apache/geode/internal/cache/control/OffHeapMemoryMonitor.java > 414036747a5062ab556b60cdb58c5ce490312c78 > geode-core/src/main/java/org/apache/geode/internal/cache/lru/EnableLRU.java > 6aaf8cc914c3b8c2e308446930bbd3215c9f58ca > > geode-core/src/main/java/org/apache/geode/internal/cache/lru/HeapLRUCapacityController.java > 5e86ce87b99fc4e8793461e881853c141fdfec1f > > geode-core/src/main/java/org/apache/geode/internal/cache/lru/LRUCapacityController.java > 3596a07df407960c3558057ac33e92c89bd5ba22 > > geode-core/src/main/java/org/apache/geode/internal/cache/lru/LRUMapCallbacks.java > 27a4ec019ae3b5fd29a26ffc451d0d9feb75f903 > > geode-core/src/main/java/org/apache/geode/internal/cache/lru/MemLRUCapacityController.java > 2c2e8ec1a805b7485e6e59858a12122fa8d88de1 > > geode-core/src/test/java/org/apache/geode/internal/cache/lru/LRUClockJUnitTest.java > 8095d5ac483be061199623a78b63618697e09d9d > > geode-core/src/test/java/org/apache/geode/internal/offheap/OffHeapLRURecoveryRegressionTest.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/58397/diff/2/ > > > Testing > ------- > > precheckin > > > Thanks, > > Darrel Schneider > >