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: [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]
