[GitHub] [spark] gengliangwang commented on pull request #36424: [SPARK-39083][CORE] : Fix race condition between update and clean app data

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

gengliangwang commented on PR #36424:
URL: https://github.com/apache/spark/pull/36424#issuecomment-1116261702

   ```
   If Thread 1 gets the lock on listing before Thread 2, isStale will be false, the app will not be removed from KVStore
   If Thread 2 gets the lock on listing before Thread 1, then isStale will be true, the app will be removed from KVStore but after that it will be added again by Thread 1.
   In both case, the entry for the application will not be deleted unexpectedly from KVStore.
   ```
   @tanvn It seems that the description doesn't match the current implementation. Can you update it?


-- 
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: reviews-unsubscribe@spark.apache.org

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