rfyu commented on code in PR #3313:
URL: https://github.com/apache/amoro/pull/3313#discussion_r1825496285


##########
amoro-ams/src/main/java/org/apache/amoro/server/AmoroServiceContainer.java:
##########
@@ -449,10 +450,11 @@ private Map<String, Object> initEnvConfig() {
     private void setIcebergSystemProperties() {
       int workerThreadPoolSize =
           Math.max(
-              Runtime.getRuntime().availableProcessors(),
+              Runtime.getRuntime().availableProcessors() / 2,

Review Comment:
   Thanks for the review. Because there used to be only one thread pool and now 
there are two. Of course, even if no changes are made, it won't cause 
significant issues.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to