beliefer commented on code in PR #39182:
URL: https://github.com/apache/spark/pull/39182#discussion_r1056127197


##########
connector/connect/common/src/main/protobuf/spark/connect/relations.proto:
##########
@@ -598,3 +595,10 @@ message ToSchema {
   // The Sever side will update the dataframe with this schema.
   DataType schema = 2;
 }
+
+// Explicitly sort the underlying plan to make the ordering deterministic for 
Sample.
+// Note: this message is only used to randomly splits DataFrame with the 
provided weights.
+message DeterministicOrder {

Review Comment:
   `DeterministicOrder` just used for explicit ordering. It means the 
underlying dataframe doesn't guarantee the ordering of rows.



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