parquet-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jul...@apache.org
Subject parquet-mr git commit: PARQUET-726: Increase max difference of testMemoryManagerUpperLimit to 10%
Date Wed, 05 Oct 2016 20:20:48 GMT
Repository: parquet-mr
Updated Branches:
  refs/heads/master b59be8659 -> 07a42d3ff


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

Author: Niels Basjes <nbasjes@bol.com>

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 <nbasjes@bol.com>
Authored: Wed Oct 5 13:20:41 2016 -0700
Committer: Julien Le Dem <julien@dremio.com>
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


Mime
View raw message