You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The text was updated successfully, but these errors were encountered:
IndexLoadingConfig
should always be initialized byTableConfig
andSchema
, 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
The text was updated successfully, but these errors were encountered: