Murtadha Hubail has posted comments on this change. Change subject: Fix Double-Checked Locking (Coverity) ......................................................................
Patch Set 1: Is this needed if all operations that might assign the index are synchronized on lcManager and they all call [ index = lcManager.getIndex(resourcePath) ] first? There is no check if the index != null before that call. -- To view, visit https://asterix-gerrit.ics.uci.edu/667 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3383035b5e8e0aa3cfd54f803f12d56090f006cf Gerrit-PatchSet: 1 Gerrit-Project: hyracks Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-HasComments: No
