tanishq-chugh commented on code in PR #5404: URL: https://github.com/apache/hive/pull/5404#discussion_r1741546961
########## 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: Yes, JDK17 requires more memory as compared to JDK8 especially due to the new module system which has been introduced as well as the [JDK-8145260](https://bugs.openjdk.org/browse/JDK-8145260) which was rightly pointed out by @Indhumathi27 . Even a lot of qtests part of our precommit tests had OOM failures when moved to jdk17 due to the same. So, this increase in tez container size is required. -- 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
