yanxiaole commented on pull request #29392: URL: https://github.com/apache/spark/pull/29392#issuecomment-672459512
Hi @zhouyejoe , Yes, you are right. I have checked the pool definition, and `checkForLogs()` and `cleanLogs()` won't run at same time, the only possible is the replay tasks. I think it's SPARK-29043 which removed the waiting to improve the performance. Does that mean I only need to synchronize the delete in `mergeApplicationListing`? ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
