singhpk234 opened a new pull request, #7353: URL: https://github.com/apache/iceberg/pull/7353
### About the change ThreadLocal is not serializable, hence using this in serializing the ObjectStoreLocationProvider makes it non-serializable which is used in creating the Serializable instance of table fail (which is used by spark to broadcast this instance to the executor, while doing read / write). Since by default ObjectLocationProvider was disabled hence this couldn't be caught by serialization tests automatically. Thanks @nastra for reporting this issue via https://github.com/apache/iceberg/issues/7345. This change makes the ThreadLocal obj transient, as well as make table serialization test run with / without obj store enabled property. cc @rdblue @jackye1995 @nastra -- 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]
