hvanhovell opened a new pull request, #49713: URL: https://github.com/apache/spark/pull/49713
This is a port of 86efd59f665d23e2dc4215d0047df43deff700a9 to branch-4.0. It includes 98957380be9856de480dadd21e4239322e199295 which fixes an issue in Catalyst with reflection.. ### What changes were proposed in this pull request? This PR makes the shared SQL (JVM) interface the primary interface for Scala/JVM based Dataframe programming. The implementations are moved to the `classic` and `connect` sub packages. The connect client had to be moved to the sql/connect/common package because serialization requires the captured client classes to be on the classpath when deserialized on the server. ### Why are the changes needed? This is the final step in creating a unified Scala interface for both Classic and Connect. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Existing tests. ### Was this patch authored or co-authored using generative AI tooling? No. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
