steveloughran commented on PR #10233: URL: https://github.com/apache/iceberg/pull/10233#issuecomment-3778390917
@danielcweeks I am revisiting this, but first in #15100 the iceberg hadoop version needs to be decreased to that of the oldest supported spark version, so as to guarantee there's nothing in iceberg-core which accidentally compiles against anything not available in spark. This will force me to return to the reflection-based binding here, but avoiding breaking iceberg on spark is critical -and independent of that API, there's a risk of some other accidental library change like an overloaded method, new constant/enum etc breaking things -- 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]
