[GitHub] [pinot] kishoreg commented on pull request #8384: Implement Native Text Operator

[GitBox <gi...@apache.org>]

kishoreg commented on pull request #8384:
URL: https://github.com/apache/pinot/pull/8384#issuecomment-1075960100


   > 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?
   
   


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org