mengna-lin commented on code in PR #3526:
URL: https://github.com/apache/parquet-java/pull/3526#discussion_r3565524175
##########
parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetWriter.java:
##########
@@ -569,6 +575,32 @@ public SELF withCompressionCodec(CompressionCodecName
codecName) {
return self();
}
+ /**
+ * Override the compression codec for a specific column.
+ *
+ * @param columnPath dot-string path of the column (e.g. {@code "my_col"})
+ * @param codecName the codec to use for that column
+ * @return this builder for method chaining.
+ */
+ public SELF withCompressionCodec(String columnPath, CompressionCodecName
codecName) {
Review Comment:
Thanks for the suggestion! My concern is that codec and level are
independent (a user may want to set only
one), and a combined method alongside the existing setters could make it
ambiguous which call wins when both configure the same column. What do you
think?
--
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]
