vvivekiyer commented on issue #12254: URL: https://github.com/apache/pinot/issues/12254#issuecomment-1889806589
I can add my understanding based on the index-spi pep doc and @gortiz can add more thoughts. tableIndexConfig is the old/current way to define (most) indexing configs. Currently, indexing config is defined in multiple places (eg Forward index is specified in FieldConfig.encoding, but can also be disabled in FieldConfig.properties.forwardIndexDisabled). More such examples are in the document - https://docs.google.com/document/d/1L_GJ2tAgZVxI_Ssa59--Mso3mK_HeAS7s7G9261-sOE/edit. Going forward, the recommendation is to use `fieldConfigList.indexes ` as the single place to store all indexing information. > but that seems error prompt to have "one take priority over another". This is done for backward compatibility till all tableConfigs are migrated to the new model of using `fieldConfigList.indexes` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
