Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/791#issuecomment-43411524
This is thread safe. `tryToPut` call `ensureFreeSpace` in a synchronized
block, so there is only one thread can run `ensureFreeSpace` at the same time,
which means each thread will select different to-be-dropped blocks. @mridulm
could you point out where may case multi thread problem that I may missed?
About exception handling, It may happen that some entries are marked as
dropping but no thread is dropping them. I will work on it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
