hvanhovell commented on code in PR #46570: URL: https://github.com/apache/spark/pull/46570#discussion_r1599956202
########## connector/connect/common/src/main/protobuf/spark/connect/base.proto: ########## @@ -199,6 +200,17 @@ message AnalyzePlanRequest { // (Required) The logical plan to get the storage level. Relation relation = 1; } + + message Checkpoint { + // (Required) The logical plan to checkpoint. + Relation relation = 1; + + // (Optional) Is it localCheckpoint Review Comment: Can the documentation be a bit less anaemic? Tell the world what this means. The same applies for the `eager` field. -- 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