pan3793 commented on code in PR #49: URL: https://github.com/apache/kyuubi-shaded/pull/49#discussion_r1721553311
########## kyuubi-relocated-spark-connect-rpc/src/main/protobuf/README.md: ########## @@ -0,0 +1,8 @@ +proto files are copied from +https://github.com/apache/spark/tree/5b2d2149b615acdd8730547a1f24c2b637222545/sql/connect/common/src/main/protobuf Review Comment: > It appears that the proto file has been forked from the spark 4.0.0-SNAPSHOT Yes, I think there will be no major changes before 4.0.0 GA, and I will make a sync after Spark 4.0.0-preview2 is available. > Which version do we intend to initially support? Spark 4.0.0. I suppose the first version will be merged into Kyuubi master after `branch-1.10` cut out, and Spark 4.0.0-preview2 or 4.0.0 will be available at that time. -- 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]
