nastra commented on PR #14590: URL: https://github.com/apache/iceberg/pull/14590#issuecomment-6600896158
> > in order for this to actually work we will still need the Spark change to land. @MonkeyCanCode is there an estimate on when that change will land in Spark? > > Hello @nastra , > > That is correct where to fix this leak, changes are needed on both Spark and Iceberg sides. If we are comfortable with this change on iceberg side, it may help spark community to adopt the current [PR](https://github.com/apache/spark/pull/59952). The current PR is under active review by cloud-fan. > > Thanks, Yong It would still be better to have the Spark side done first, otherwise we're adding close behavior in Iceberg that is never called. -- 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]
