cloud-fan commented on code in PR #52256:
URL: https://github.com/apache/spark/pull/52256#discussion_r2361526574
##########
sql/connect/common/src/main/protobuf/spark/connect/base.proto:
##########
@@ -1095,6 +1095,32 @@ message FetchErrorDetailsResponse {
// Portable error identifier across SQL engines
// If null, error class or SQLSTATE is not set.
optional string sql_state = 4;
+
+ // Additional information if the error was caused by a breaking change.
+ optional BreakingChangeInfo breaking_change_info = 5;
+ }
+
+ // BreakingChangeInfo defines the schema for breaking change information.
+ message BreakingChangeInfo {
+ // A message explaining how the user can migrate their job to work
+ // with the breaking change.
+ repeated string migration_message = 1;
Review Comment:
It's a string list in JSON because we don't want to have super long lines in
the JSON file. It's not really about semantic and I think we should not inherit
this trick in the protobuf message.
--
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
