amaliujia commented on code in PR #38604:
URL: https://github.com/apache/spark/pull/38604#discussion_r1019878719
##########
connector/connect/src/main/scala/org/apache/spark/sql/connect/service/SparkConnectStreamHandler.scala:
##########
@@ -50,9 +49,9 @@ class SparkConnectStreamHandler(responseObserver:
StreamObserver[Response]) exte
def handlePlan(session: SparkSession, request: proto.Request): Unit = {
Review Comment:
@cloud-fan
From what I can tell in the near term we probably won't touch `Command`
proto again. How about let us wait for some time to see if there is any new
need that actually needs a separate `Command` proto. If not we can deprecate it
and merge with `Relation` proto?
--
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
