Jackie-Jiang opened a new issue, #14106: URL: https://github.com/apache/pinot/issues/14106
`IndexLoadingConfig` should always be initialized by `TableConfig` and `Schema`, and all the index related config should be extracted from them. Right now we allow setters to internal fields such as `_invertedIndexColumns`, `noDictionaryColumns` etc which makes config management extremely hard, especially after introducing Index SPI. We have run into problems such as forward index config not picked up from table config. We use this issue to track all effort of removing setters from `IndexLoadingConfig`. Similarly, we should also remove all setters in `SegmentGeneratorConfig` -- 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.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
