kishoreg commented on a change in pull request #3852: add support for parquet
reader
URL: https://github.com/apache/incubator-pinot/pull/3852#discussion_r257923979
##########
File path: pinot-core/pom.xml
##########
@@ -213,5 +213,32 @@
<groupId>org.apache.commons</groupId>
<artifactId>commons-math</artifactId>
</dependency>
+
+ <dependency>
+ <groupId>org.apache.parquet</groupId>
+ <artifactId>parquet-avro</artifactId>
+ <version>${parquet.version}</version>
+ </dependency>
+
+ <dependency>
Review comment:
It's probably ok for now. I cant think of any other work around for now.
the hadoop-client dependency in common was added because of hdfs support I
think. We plan to remove it once we have the support for deep storage and Uber
folks stop using that feature. Adding more code that depends on this will make
it harder for us to remove the dependency on hadoop client.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
