BsoBird commented on code in PR #5404: URL: https://github.com/apache/hive/pull/5404#discussion_r1897053638
########## data/conf/llap/hive-site.xml: ########## @@ -49,7 +49,7 @@ <!-- the Vertex resource, so this has to be defined in order to get consistent test results --> <property> <name>hive.tez.container.size</name> - <value>128</value> + <value>256</value> Review Comment: @tanishq-chugh @okumin @abstractdog @aturoczy @deniskuzZ @Aggarwal-Raghav Hello. We use JDK21 to run HIVE+TEZ in our downstream environment. After adopting JDK21, I observed a significant increase in memory consumption by TEZ. For TEZ container sizes ranging from 4GB to 8GB, I had to lower the value of tez.runtime.shuffle.merge.percent from the default 0.9 to 0.75 to successfully execute the jobs. If I run the job with the default value of 0.9, I won't even be able to successfully run any Tez container. Thus, there might be additional underlying issues; the mere existence of [JDK-8145260](https://bugs.openjdk.org/browse/JDK-8145260) seems insufficient to account for this problem entirely. -- 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
