yahoNanJing commented on code in PR #221:
URL: https://github.com/apache/arrow-ballista/pull/221#discussion_r972543545
##########
ballista/rust/core/proto/ballista.proto:
##########
@@ -534,6 +534,15 @@ message PartitionId {
uint32 partition_id = 4;
}
+// Unique identifier for a materialized partition of data
+message PartitionIds {
+ string job_id = 1;
+ uint32 stage_id = 2;
+ repeated uint32 partition_id = 3;
+ // Output partition for shuffle writer
+ repeated PhysicalHashRepartition output_partitioning = 4;
Review Comment:
Better to align with the single task definition and move output_partitioning
to the *MultiTaskDefinition*.
--
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]
