Till Westmann has posted comments on this change. Change subject: Fulltext search initial implementation ......................................................................
Patch Set 14: Looking at this for the first time, I've got 3 high level questions: 1) Is it really necessary to change every line of asterixdb/asterix-app/data/csv/fragile_02.adm? 2) It seems that there are a number of quite reasonable SonarQube comments (not all are ...). Why didn't you address them? 3) Why is FULLTEXT_CONTAINS added to AlgebricksBuiltinFunctions? It seems that it's never used in Algebricks.. -- To view, visit https://asterix-gerrit.ics.uci.edu/989 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a Gerrit-PatchSet: 14 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Taewoo Kim <[email protected]> Gerrit-Reviewer: Heri Ramampiaro <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Jianfeng Jia <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Taewoo Kim <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-HasComments: No
