siddharthteotia commented on a change in pull request #6876:
URL: https://github.com/apache/incubator-pinot/pull/6876#discussion_r626081079
##########
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/TableConfigUtils.java
##########
@@ -573,6 +573,9 @@ private static void validateFieldConfigList(@Nullable
List<FieldConfig> fieldCon
Preconditions.checkArgument(!noDictionaryColumns.contains(columnName),
"FieldConfig encoding type is different from indexingConfig
for column: " + columnName);
}
+
Preconditions.checkArgument(fieldConfig.getNoDictionaryColumnCompressorCodec()
== null,
+ "FieldConfig column compression codec is only supported for
single value raw encoding type");
Review comment:
(nit) Also add "Set compression codec to null for dictionary encoding
type"
--
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.
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
