adutra commented on PR #1880: URL: https://github.com/apache/polaris/pull/1880#issuecomment-2970925926
> Not against this PR, but wouldn't it be better to not even have those dependencies that bring relocated code or not have dependencies on uber jars at all, as long as there are alternatives? Well, we certainly could try to ban the iceberg shaded dependencies, but afair the Spark Client module is fairly complex in terms of dependency management. And as for Testcontainers, I think that if your tests require testcontainers, you get the testcontainers shaded dependencies (e.g. Awaitility) "for free" 🤷♂️ – and there is no way to exclude them since they are in the same jar. -- 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]
