jerolba commented on PR #3781: URL: https://github.com/apache/parquet-java/pull/3781#issuecomment-5592508837
I agree that it would be nice to have it tested, but it will add a lot of complexity to the project just for a test setup. I've tested this change in [Carpet](https://github.com/jerolba/parquet-carpet/) removing `hadoop-mapreduce-client-core` [dependency](https://github.com/jerolba/parquet-carpet/blob/master/carpet-record/build.gradle#L27) and, based on the code path used by the library, it works. The change doesn't ensure that usages of other `parquet-hadoop` classes will not require the dependency. Unfortunately, we are far from removing the dependency of `hadoop-common`. I think that it will require a complete rewrite of the `parquet-hadoop` module. -- 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]
