Jackie-Jiang commented on pull request #8384: URL: https://github.com/apache/pinot/pull/8384#issuecomment-1076572595
> > Let's not merge this PR yet. Want to ensure all the functionalities we want to support for text index is covered by the index (positional info needs to be stored). The actual support can be added in multiple PRs, but we don't want to change index format frequently. > > You are right if we try to change text match operator semantics.. I was thinking more along using like operator to use this index and leave the text match operator as it is. Phrase match is not that important and can be done later in another PR > > Wdyt? We don't need this index in order to support LIKE. LIKE and regexpLike is already supported with the FST (either lucene or native) and won't use this index. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
