zabetak opened a new pull request, #4792:
URL: https://github.com/apache/hive/pull/4792

   ### What changes were proposed in this pull request?
   Increase max heap space for Tez Application master to 512MB.
   
   ### Why are the changes needed?
   java.lang.OutOfMemoryError: GC overhead limit exceeded is thrown by the Tez 
Application Master (AM) cause the current heap size (128MB) is not enough to 
accommodate the needs of multiple Tez containers running. Each running 
container requires roughly 10MB of memory in the AM. The AM 
accumulates/manipulates multiple configuration objects (some of them retaining 
as much as 1MB of heap) per container. The heap gradually becomes full and GC 
is spending a lot of CPU time to clean things up without really making much 
progress since containers are reused and heap cannot shrink.
   
   Extended analysis and justification under HIVE-27695.
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### Is the change a dependency upgrade?
   No
   
   ### How was this patch tested?
   ```
   mvn test -Dtest=TestMiniTezCliDriver
   ```
   Running all tests for `TestMiniTezCliDriver` without the changes here leads 
to OOM 95% of the time.


-- 
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: gitbox-unsubscr...@hive.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to