zentol commented on a change in pull request #19083:
URL: https://github.com/apache/flink/pull/19083#discussion_r826852181
##########
File path: docs/content/docs/connectors/datastream/formats/parquet.md
##########
@@ -39,46 +39,71 @@ To use the format you need to add the Flink Parquet
dependency to your project:
<version>{{< version >}}</version>
</dependency>
```
-
+
+For reading Avro records, parquet-avro dependency is required additionally:
+
+```xml
+<dependency>
+ <groupId>org.apache.parquet</groupId>
+ <artifactId>parquet-avro</artifactId>
+ <version>${flink.format.parquet.version}</version>
+ <optional>true</optional>
+ <exclusions>
+ <exclusion>
+ <groupId>org.apache.hadoop</groupId>
+ <artifactId>hadoop-client</artifactId>
+ </exclusion>
+ <exclusion>
+ <groupId>it.unimi.dsi</groupId>
+ <artifactId>fastutil</artifactId>
+ </exclusion>
+ </exclusions>
Review comment:
As for the hadoop-client exclusion in flink-parquet, it's a bit odd, and
the only explanation I can offer is that it exists for dependency convergence.
An exclusion in the user application somewhat makes sense because we don't
want it to be bundled, but imo it would be more appropriate to set it to
provided somehow.
--
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]
