Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MockBatchDeleter ConcurrentModificationException #3

Open
kvangrae opened this issue Mar 1, 2013 · 1 comment
Open

MockBatchDeleter ConcurrentModificationException #3

kvangrae opened this issue Mar 1, 2013 · 1 comment
Labels
bug

Comments

@kvangrae
Copy link
Contributor

kvangrae commented Mar 1, 2013

In the MockBatchDeleter, occasionally it seems with MapReduce tests, was throwing ConcurrentModificationException when purging the MockInstance. This is because the mock instance is backed by TreeMap. Without looking into this deeper, what probably needs to happen is that we should swap out the TreeMap with some sort of ConcurrentTreeMap. Java has a ConcurrentHashMap, but we need to support natural ordering via a TreeMap.

In the meantime, the MockBatchDeleter catches the ConcurrentModificationException and ignores it. It seems to work fine for now, but may present an issue in the future.
@kvangrae
Copy link
Contributor Author

kvangrae commented Mar 2, 2013

I looked at MockTable and it uses a ConcurrentSkipListMap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kvangrae