rdblue commented on pull request #1778: URL: https://github.com/apache/iceberg/pull/1778#issuecomment-730059373
> Shall we be using SessionConfigSupport in Spark for this? Does `SessionConfigSupport` work with catalogs? I thought it was only supported in some `DataSource` paths. I typically use catalog configuration, which I think cleanly removes the need for that interface. That may be a clean way to plug in here, I don't really have a preference. @samarthjain, I think you probably want to update the Spark 3 path as well, since that's what we use in 2.4.4. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
