jackye1995 commented on pull request #3270: URL: https://github.com/apache/iceberg/pull/3270#issuecomment-987325605
We only have 1 bundled jar for an engine for all JDK versions, so we have to either include all the dependencies, or no dependency at all. I think the reason we did not include the jars for AWS was because we didn't want to inflate the bundled jar size. That's something we also need to consider here. For Aliyun, I am checking the maven repo, to include all dependencies for OSS, it also needs 3 other direct dependencies (core, ram, kms), all their dependencies, and the 3 javax jars. The runtime jar of jaxb was over 1MB. So it's not a small dependency footprint. I have not checked if the jars conflict with any jars in the engine, that's something we also need to be careful. Plus it also creates an inconsistent experience across intergrated services, so I am a bit hesitated here. @rdblue any thoughts? -- 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]
