asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Luo Chen (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: [ASTERIXDB-2203][IDX] Eliminate sort operator for secondary ...
Date Thu, 21 Dec 2017 21:57:50 GMT
Luo Chen has posted comments on this change.

Change subject: [ASTERIXDB-2203][IDX] Eliminate sort operator for secondary primary index
- user model changes: no - storage format changes: no - interface changes: no
......................................................................


Patch Set 1:

Just copied what I replied to Ali here:

For the secondary primary index, I don't think the correlated merge policy make sense, as
there is no need to search from secondary primary index and then go back to the primary index.
In the contrast, it's better for the secondary primary index to have fewer components as possible.
Thus, it's safe to ignore this correlated option for secondary primary index and always fall
back to the old one.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc4f90928250aafd90d965028beb07561222086c
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman <ali.al.solaiman@gmail.com>
Gerrit-Reviewer: Ali Alsuliman <ali.al.solaiman@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Luo Chen <cluo8@uci.edu>
Gerrit-Reviewer: Murtadha Hubail <mhubail@apache.org>
Gerrit-HasComments: No

Mime
View raw message