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 <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Luo Chen <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-HasComments: No
