----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44869/#review123898 -----------------------------------------------------------
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/control/HeapMemoryMonitor.java (line 458) <https://reviews.apache.org/r/44869/#comment186190> I think the correct test is "!= -1" instead of "> -1". - Darrel Schneider On March 16, 2016, 10:43 a.m., Sai Boorlagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44869/ > ----------------------------------------------------------- > > (Updated March 16, 2016, 10:43 a.m.) > > > Review request for geode, Darrel Schneider, Eric Shu, and Swapnil Bawaskar. > > > Bugs: GEODE-1096 > https://issues.apache.org/jira/browse/GEODE-1096 > > > Repository: geode > > > Description > ------- > > Fixed updateStateAndSendEvent to consider testBytesUsedForThresholdSet > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/control/HeapMemoryMonitor.java > 27a6dffe8d41bbb3dbe44700e4980b513bb07121 > > Diff: https://reviews.apache.org/r/44869/diff/ > > > Testing > ------- > > precheckin and regression tests > > > Thanks, > > Sai Boorlagadda > >
