siddharthteotia commented on code in PR #9454:
URL: https://github.com/apache/pinot/pull/9454#discussion_r980314842
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/loader/IndexLoadingConfig.java:
##########
@@ -215,6 +218,26 @@ private void extractFromTableConfig(TableConfig
tableConfig) {
}
}
+ /**
+ * Extracts compressionType for each column. Populates a map containing
column name as key and compression type as
+ * value. Note that only RAW forward index columns will be populated in this
map.
+ * @param tableConfig table config
+ */
+ private void extractCompressionConfigs(TableConfig tableConfig) {
Review Comment:
I think we need to handle the existing way compression is configured. There
could be cases where people are configuring compressionCodec without going via
FieldConfig (since this was added later and no migration hasn't happened).
--
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]
