agrawaldevesh edited a comment on pull request #29014:
URL: https://github.com/apache/spark/pull/29014#issuecomment-665168394


   > @agrawaldevesh we are really close, sorry for being this picky (today) but 
I am sure this extra care will serve us well.
   
   @attilapiros Always always appreciated !. Better to be nitpicky once than 
confuse a thousand future readers :-P
   
   I have reworked the whole config confusion in the test. Now there are only 
the following configs:
   - Master and worker config (that is used to create the master and workers 
before each test)
   - SparkConf proper for the spark job and all of the driver/scheduler related 
config.
   - Default config used for the external shuffle service
   
   This has really cleaned up the config mess so thanks for spotting this.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to