Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index ......................................................................
Patch Set 3: (2 comments) @Dmitry: thanks. I have addressed your comments. https://asterix-gerrit.ics.uci.edu/#/c/2854/3/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java File asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java: Line 1205: for (LogicalVariable v : usedVarsAfterTopOp) { > This loop can be replaced by varsUsedInTopOpButNotAfterwards.removeAll(used Done https://asterix-gerrit.ics.uci.edu/#/c/2854/3/asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index/btree-sidx-idxonly-10.sqlpp File asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index/btree-sidx-idxonly-10.sqlpp: Line 48: statues_count: int64 > should be statuses_count here and everywhere below Done -- To view, visit https://asterix-gerrit.ics.uci.edu/2854 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idcc4cbe08323e0c6edb4a01637b2017128da1ab5 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Taewoo Kim <wangs...@gmail.com> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin <dmitry.lycha...@couchbase.com> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Taewoo Kim <wangs...@gmail.com> Gerrit-HasComments: Yes