maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2075542597


##########
tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/impl/TestSimpleFetchedInputAllocator.java:
##########
@@ -66,7 +66,7 @@ public void testInMemAllocation() throws IOException {
     
     
     FetchedInput fi2 = inputManager.allocate(requestSize, compressedSize, new 
InputAttemptIdentifier(2, 1));
-    assertEquals(FetchedInput.Type.MEMORY, fi2.getType());
+    assertEquals(FetchedInput.Type.DISK, fi2.getType());

Review Comment:
   This test failed due to when i increased memory of surefire "-Xmx4096m 
-XX:+HeapDumpOnOutOfMemoryError". Anyway this value i will revert and check 
once. This value i modified as compile and run with jdk17 it is taking more 
time for some of the tests and some tests are timeout. I will revert this value 
to "xmx1024m" and check once. if any no issue then i revert this value and 
modify the test case accordingly. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@tez.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to