techaddict commented on code in PR #39110:
URL: https://github.com/apache/spark/pull/39110#discussion_r1057953155


##########
core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto:
##########
@@ -390,3 +390,38 @@ message SQLExecutionUIData {
   repeated int64 stages = 11;
   map<int64, string> metric_values = 12;
 }
+
+message RDDOperationEdge {
+  int32 from_id = 1;
+  int32 to_id = 2;
+}
+
+enum DeterministicLevel {
+  DETERMINATE = 0;

Review Comment:
   makes sense, but getting this error,
   > store_types.proto:400:3: "UNSPECIFIED" is already defined in 
"org.apache.spark.status.protobuf".
   > 
   > 
   > store_types.proto:400:3: Note that enum values use C++ scoping rules, 
meaning that enum values are siblings of their type, not children of it.  
Therefore, "UNSPECIFIED" must be unique within 
"org.apache.spark.status.protobuf", not just within "DeterministicLevel".
   



-- 
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]

Reply via email to