Repository: parquet-mr
Updated Branches:
  refs/heads/master b59be8659 -> 07a42d3ff


PARQUET-726: Increase max difference of testMemoryManagerUpperLimit to 10%

Author: Niels Basjes <[email protected]>

Closes #370 from nielsbasjes/PARQUET-726 and squashes the following commits:

f385ede [Niels Basjes] PARQUET-726: Increase max difference of 
testMemoryManagerUpperLimit to 10%


Project: http://git-wip-us.apache.org/repos/asf/parquet-mr/repo
Commit: http://git-wip-us.apache.org/repos/asf/parquet-mr/commit/07a42d3f
Tree: http://git-wip-us.apache.org/repos/asf/parquet-mr/tree/07a42d3f
Diff: http://git-wip-us.apache.org/repos/asf/parquet-mr/diff/07a42d3f

Branch: refs/heads/master
Commit: 07a42d3ffd034e467e49b5c449d4f5f81c471cc5
Parents: b59be86
Author: Niels Basjes <[email protected]>
Authored: Wed Oct 5 13:20:41 2016 -0700
Committer: Julien Le Dem <[email protected]>
Committed: Wed Oct 5 13:20:41 2016 -0700

----------------------------------------------------------------------
 .../java/org/apache/parquet/hadoop/TestMemoryManager.java     | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/parquet-mr/blob/07a42d3f/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestMemoryManager.java
----------------------------------------------------------------------
diff --git 
a/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestMemoryManager.java 
b/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestMemoryManager.java
index b91fedd..3e7e3d7 100644
--- 
a/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestMemoryManager.java
+++ 
b/parquet-hadoop/src/test/java/org/apache/parquet/hadoop/TestMemoryManager.java
@@ -68,10 +68,11 @@ public class TestMemoryManager {
   public void testMemoryManagerUpperLimit() {
     // Verify the memory pool size
     // this value tends to change a little between setup and tests, so this
-    // validates that it is within 5% of the expected value
+    // validates that it is within 10% of the expected value
     long poolSize = 
ParquetOutputFormat.getMemoryManager().getTotalMemoryPool();
-    Assert.assertTrue("Pool size should be within 5% of the expected value",
-        Math.abs(expectedPoolSize - poolSize) < (long) (expectedPoolSize * 
0.05));
+    Assert.assertTrue("Pool size should be within 10% of the expected value" +
+                    " (expected = " + expectedPoolSize + " actual = " + 
poolSize + ")",
+        Math.abs(expectedPoolSize - poolSize) < (long) (expectedPoolSize * 
0.10));
   }
 
   @Test

Reply via email to