[GitHub] [incubator-pinot] jackjlli commented on pull request #6070: Add Hadoop related dependencies in pinot-tool module

2020-09-30 Thread GitBox
jackjlli commented on pull request #6070: URL: https://github.com/apache/incubator-pinot/pull/6070#issuecomment-701903297 Discussed with @mayankshriv. The issue before this PR is that pinot-orc and pinot-parquet module needs Hadoop libraries. While the Hadoop dependencies are in provided

[GitHub] [incubator-pinot] jackjlli commented on pull request #6070: Add Hadoop related dependencies in pinot-tool module

2020-09-29 Thread GitBox
jackjlli commented on pull request #6070: URL: https://github.com/apache/incubator-pinot/pull/6070#issuecomment-700932259 @kishoreg we do use `PluginManager` to instantiate the specific record reader. It's just that the required Hadoop jars are not pulled in because they are transitive

[GitHub] [incubator-pinot] jackjlli commented on pull request #6070: Add Hadoop related dependencies in pinot-tool module

2020-09-29 Thread GitBox
jackjlli commented on pull request #6070: URL: https://github.com/apache/incubator-pinot/pull/6070#issuecomment-700930974 @mayankshriv the Hadoop jars are also needed for parquet as well. This is an automated message from