Github user mridulm commented on the pull request:
https://github.com/apache/spark/pull/791#issuecomment-43413071
Use of dropping is not my safe
On 17-May-2014 9:42 pm, "Wenchen Fan" <[email protected]> wrote:
> 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 <https://github.com/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.
>
> â
> Reply to this email directly or view it on
GitHub<https://github.com/apache/spark/pull/791#issuecomment-43411524>
> .
>
---
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.
---
