Alawn/Chronicle-Map
1
0
Fork 0
mirror of https://github.com/OpenHFT/Chronicle-Map.git synced 2021-09-13 01:26:12 +00:00
Code Releases Activity

Address the issue with ChronicleHash accessors throwing AssertionError instead of IllegalStateException after ChronicleHash is closed

Browse Source
This commit is contained in:
leventov
2016-08-05 23:31:09 +07:00
parent ba8227ade1
commit 9a5d6fa50e
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/net/openhft/chronicle/hash/impl/stage/hash/ThreadLocalState.java
+5 -3
View File
@@ -55,8 +55,8 @@ public abstract class ThreadLocalState {
* is a nested context.
*/
public boolean lockContextLocally() {
// h().isOpen() check guarantees no starvation of a thread calling chMap.close() and trying
// to close this context by closeContext() method below, while the thread owning this
// hash().isOpen() check guarantees no starvation of a thread calling chMap.close() and
// trying to close this context by closeContext() method below, while the thread owning this
// context frequently locks and unlocks it (e. g. in a loop). This is also the only check
// for chMap openness during the whole context usage lifecycle.
if (hash().isOpen() && MEMORY.compareAndSwapInt(this, CONTEXT_LOCK_OFFSET,
@@ -65,7 +65,9 @@ public abstract class ThreadLocalState {
} else {
if (contextLock == CONTEXT_LOCKED_LOCALLY)
return false;
if (contextLock == CONTEXT_CLOSED)
// Don't extract this hash().isOpen() and the one above, because they could different
// results (the first (above) could return true, the second (below) - false).
if (!hash().isOpen())
throw new ChronicleHashClosedException();
throw new AssertionError("Unknown context lock state: " + contextLock);
}