gephi / graphstore / #610

92%

push

Avoid taking the graph lock while holding the table lock or view index monitor (#319)

* Avoid taking the graph lock while holding the table lock or view index monitor

TimeIndexStore.remove(K, Element), IndexStore.set and IndexStore.clear(T)
checked view membership through the graph decorator, which takes the graph
read lock. With a view index present, removing a timestamp or setting an
indexed attribute could deadlock against a thread holding the graph write
lock and waiting for the table lock or monitor. They now read the view
directly, like TimeIndexStore.add. IndexStore.deleteViewIndex also removes
under the view index monitor that set() iterates under.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Check the view index in the timestamp test and cover IndexStore.clear(T)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

9 of 9 new or added lines in 2 files covered. (100.0%)

12087 of 13203 relevant lines covered (91.55%)

0.92 hits per line