[GitHub] [pinot] gortiz commented on pull request #10191: [Index SPI] IndexType

["gortiz (via GitHub)" <gi...@apache.org>]

gortiz commented on PR #10191:
URL: https://github.com/apache/pinot/pull/10191#issuecomment-1449772728

   > Just realize this is copy-pasted from the class in segment local, but the original IndexHandler is modified recently..
   Let's directly move the class to segment spi since there are very few usages of 
   
   I've just checked and there is no recent change in the interface. Both interfaces are not the same because I removed `IndexCreatorProvider`, as it is not needed in this version.
   
   I'm going to move the class. It would increase the number of changes in this PR, but they should only be imports.


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