Jackie-Jiang commented on PR #16103: URL: https://github.com/apache/pinot/pull/16103#issuecomment-2977711176
cc @gortiz I think the overall direction should be to move all index related configs to `FieldConfig`. Currently the only exception is star-tree index, and this is adding one more exception. Can you share what functionality is not supported with the new multi-column text index? Maybe I'm missing something, but from my point of view, this is similar to the current single file approach which encapsulate all indexes except for star-tree index and text index, and we are essentially adding one more single file to encapsulate all text indexes. For now, we should at least add some validation to prevent user from configuring the same column in both places and result in 2 copies of index being generated. -- 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
