[parquet-mr] branch master updated: PARQUET-1971: Further increase max difference of testMemoryManagerUpperLimit to 15% (#863)

[gabor]

This is an automated email from the ASF dual-hosted git repository.

gabor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-mr.git


The following commit(s) were added to refs/heads/master by this push:
     new e7f9a66  PARQUET-1971: Further increase max difference of 
testMemoryManagerUpperLimit to 15% (#863)
e7f9a66 is described below

commit e7f9a666f1c55bd4084d49f027eecb745f8fe5ff
Author: Gabor Szadovszky <ga...@apache.org>
AuthorDate: Thu Feb 4 13:10:02 2021 +0100

    PARQUET-1971: Further increase max difference of 
testMemoryManagerUpperLimit to 15% (#863)
---
 .../java/org/apache/parquet/hadoop/TestMemoryManager.java    | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

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 836ca6d..21f279b 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
@@ -1,4 +1,4 @@
-/* 
+/*
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements.  See the NOTICE file
  * distributed with this work for additional information
@@ -6,9 +6,9 @@
  * to you under the Apache License, Version 2.0 (the
  * "License"); you may not use this file except in compliance
  * with the License.  You may obtain a copy of the License at
- * 
+ *
  *   http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing,
  * software distributed under the License is distributed on an
  * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
@@ -68,11 +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 10% of the expected value
+    // validates that it is within 15% of the expected value
     long poolSize = 
ParquetOutputFormat.getMemoryManager().getTotalMemoryPool();
-    Assert.assertTrue("Pool size should be within 10% of the expected value" +
+    Assert.assertTrue("Pool size should be within 15% of the expected value" +
                     " (expected = " + expectedPoolSize + " actual = " + 
poolSize + ")",
-        Math.abs(expectedPoolSize - poolSize) < (long) (expectedPoolSize * 
0.10));
+        Math.abs(expectedPoolSize - poolSize) < (long) (expectedPoolSize * 
0.15));
   }
 
   @Test