You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by sa...@apache.org on 2016/03/01 02:59:34 UTC

incubator-geode git commit: added freeMemory assert

Repository: incubator-geode
Updated Branches:
  refs/heads/feature/GEODE-996 367011540 -> 68014481f


added freeMemory assert


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/68014481
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/68014481
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/68014481

Branch: refs/heads/feature/GEODE-996
Commit: 68014481ff5d4e6f85602f331a9f6f3d118ab7ce
Parents: 3670115
Author: Sai Boorlagadda <sb...@pivotal.io>
Authored: Mon Feb 29 17:59:22 2016 -0800
Committer: Sai Boorlagadda <sb...@pivotal.io>
Committed: Mon Feb 29 17:59:22 2016 -0800

----------------------------------------------------------------------
 .../com/gemstone/gemfire/internal/offheap/FreeListManager.java  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/68014481/geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/FreeListManager.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/FreeListManager.java b/geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/FreeListManager.java
index 4ecc4e4..3f7035c 100644
--- a/geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/FreeListManager.java
+++ b/geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/FreeListManager.java
@@ -490,9 +490,10 @@ public class FreeListManager {
         //free memory is available as one fragment, so no fragmentation
         return 0;
       } else {
-        //when more than 1 fragment is available
-        //then compare the no. of available fragments with max no. of possible fragments
+        //more than 1 fragment is available so freeMemory is > 0
+        //compare the no. of available fragments with max no. of possible fragments
         long freeMemory = getFreeMemory();
+        assert freeMemory > 0;
         long maxPossibleFragments = freeMemory / ObjectChunk.MIN_CHUNK_SIZE;
         double fragmentation = ((double) availableFragments /(double) maxPossibleFragments) * 100d;
         return (int) Math.rint(fragmentation);