manuzhang commented on PR #11157: URL: https://github.com/apache/iceberg/pull/11157#issuecomment-2443260806
@RussellSpitzer I can revert to previous commit and this is Spark specific, but can you elaborate on why `LazyExecutorService` is better than`SerializableSupplier`? I agree with you that people can pass around a non-lazy ExecutorService in future implementations. I submitted #11417 to add warning in the doc since this PR can't get into 1.7.0 -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
