You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/06/10 09:25:02 UTC

[GitHub] [spark] maropu commented on pull request #28778: [SPARK-31949][SQL] Add spark.default.parallelism in SQLConf for isolated across session

maropu commented on pull request #28778:
URL: https://github.com/apache/spark/pull/28778#issuecomment-641873870


   Yea, having a sessein-local default parallelism param in the SQL side looks fine to me. But, as @cloud-fan said above, you need more work for applying the param into the exising SQL logics.


----------------------------------------------------------------
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.

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



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