SCHJonathan commented on code in PR #52154:
URL: https://github.com/apache/spark/pull/52154#discussion_r2371688131
##########
sql/connect/common/src/main/protobuf/spark/connect/pipelines.proto:
##########
@@ -101,11 +121,23 @@ message PipelineCommand {
// Name of the dataset this flow writes to. Can be partially or fully
qualified.
optional string target_dataset_name = 3;
- // An unresolved relation that defines the dataset's flow.
- optional spark.connect.Relation relation = 4;
+ // [Deprecated] An unresolved relation that defines the dataset's flow.
+ optional spark.connect.Relation relation = 4 [deprecated = true];
Review Comment:
Since SDP currently still extensive using this field right now, wdyt I mark
the proto as deprecated in this PR, followup with a PR updating the code usage
to use the new proto. After that, we remove the deprecated 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: [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]
