Yingyi Bu has posted comments on this change. Change subject: Range connector updates to support interval partitioning. ......................................................................
Patch Set 1: Hi Preston, Here are two high-level comments: 1. Based on my rough memory, we decided to not implement new hyracks connectors for range partitioning, but maps each tuple to partition(s) before hash partitioning, similarly to tokenizing, did we? 2. I propose to postpone the merge of this change util we have automatic runtime range generation in the master. It shouldn't be too distant since James is working on that? IMO, RangeMap is fine for research and experiments but seems to have no reason to stay in the master. Since all the code that deals with RangeMap will go away by the time we have runtime range generation, it would be better to not add it now and fix it later. @Till, thoughts? Best, Yingyi -- To view, visit https://asterix-gerrit.ics.uci.edu/803 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I551e3196d8a101cf94c084c14842aa1af11632ce Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Preston Carman <prest...@apache.org> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Till Westmann <ti...@apache.org> Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com> Gerrit-HasComments: No
