panbingkun commented on PR #47515: URL: https://github.com/apache/spark/pull/47515#issuecomment-2255590695
> a potential followup of this patch, the deprecated method should have a default implementation or simply be deleted, otherwise each connectors have to implement the deprecated method, which does not make sense. https://github.com/unitycatalog/unitycatalog/blob/0de24a0cfac5b40316a97092dbfb769218602720/connectors/spark/src/main/scala/io/unitycatalog/connectors/spark/UCSingleCatalog.scala#L160 I think we can follow the following pattern to get it done. Before that, we need to get support from @cloud-fan , and I will continue. https://github.com/apache/spark/pull/45368 -- 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]
