[GitHub] caneGuy edited a comment on issue #23519: [SPARK-26601][SQL] Make broadcast-exchange thread pool configurable

[GitBox <gi...@apache.org>]

caneGuy edited a comment on issue #23519: [SPARK-26601][SQL] Make broadcast-exchange thread pool configurable
URL: https://github.com/apache/spark/pull/23519#issuecomment-453822112
 
 
   > > But some times, if the Thead object do not GC quickly it may caused server(driver) OOM. In such case,we need to make this thread pool configurable.
   > 
   > So sounds you want to lower the number of threads? By doing this, don't you also decrease parallelism and have longer waiting for other broadcasting?
   
   @viirya thanks,for some long running thriftServer  shows that we can lower this number to keep the server's high availability.If we use default number 128, we may encounter an OOM error and currently we can just turn off the broadcast join to fix this memory error.Make this number configurable will give users a more useful option.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org