[GitHub] [pinot] mayankshriv commented on pull request #7405: Introduce Native Text Indices (Core Functionality)

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

mayankshriv commented on pull request #7405:
URL: https://github.com/apache/pinot/pull/7405#issuecomment-917129393


   > > I would suggest creating a new pinot-fst module for the new FST implementation.
   > 
   > Not sure about this.. we don't create modules at root level for other indexing. Need to think carefully before creating modules at the root.. at some point, ideally, we should create plugin mechanisms for indexes and then create a module for each index.. we are not there yet
   
   +1 to Kishore, we don't have (or create) new modules at root level for other indexing either.


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