rdblue commented on a change in pull request #1316:
URL: https://github.com/apache/iceberg/pull/1316#discussion_r468771458
##########
File path: core/src/main/java/org/apache/iceberg/avro/Avro.java
##########
@@ -82,11 +96,11 @@ public static WriteBuilder write(OutputFile file) {
public static class WriteBuilder {
private final OutputFile file;
+ private final Map<String, String> config = Maps.newHashMap();
Review comment:
That's correct right now. We have been gradually expanding the settings
that are supported in Parquet and I expect that to happen here, too. For
example, when we add Zstd support for Avro, we will probably also want to
support the compression level setting. Parquet also copies config into the
Hadoop `Configuration`, which we could do for Avro if there is a use case.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
