Ian Maxon has posted comments on this change. Change subject: [ASTERIXDB-1946][STO][IDX]Create RTree&InvertedIdx for Correlated Datasets ......................................................................
Patch Set 4: > > Why's the BinaryTokenizer have to be changed? > > The previous BinaryTokenizer assumes the docField must be string > type. However, for the inverted index, if a tuple is an anti-matter > tuple, we can set the secondary field (i.e., the docField) as > missing type so that the secondary field needs not to be tokenized. > This is not supported by the current BinaryTokenizer, and thus some > simple changes are made to it to support this. I see, that makes sense. Thanks for the explanation -- To view, visit https://asterix-gerrit.ics.uci.edu/1845 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I100fc0b86b8a6fa36a95d77806107bad0307544e Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Luo Chen <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Jianfeng Jia <[email protected]> Gerrit-Reviewer: Luo Chen <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: No
