asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Taewoo Kim (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Index-Only Plan Patch Set 4: modifies SearchOpCallBack
Date Tue, 28 Mar 2017 21:24:01 GMT
Taewoo Kim has posted comments on this change.

Change subject: Index-Only Plan Patch Set 4: modifies SearchOpCallBack
......................................................................


Patch Set 5:

Sorry. One more: the current implementation works since we use instantTryLock. Even if we
see an antimatter tuple, we don't do anything since no lock is being hold in the proceed().
There's nothing to cancel. So, it works, but it's not the correct implementation.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1597
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I16992b3bdc1bfc739417d11366d6bafc59294a0b
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim <wangsaeu@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Taewoo Kim <wangsaeu@gmail.com>
Gerrit-Reviewer: Yingyi Bu <buyingyi@gmail.com>
Gerrit-HasComments: No

Mime
View raw message