avantgardnerio commented on code in PR #269:
URL: https://github.com/apache/arrow-ballista/pull/269#discussion_r979285047


##########
ballista/rust/core/proto/ballista.proto:
##########
@@ -517,6 +517,8 @@ message FetchPartition {
   uint32 stage_id = 2;
   uint32 partition_id = 3;
   string path = 4;
+  string host = 5;
+  uint32 port = 6;

Review Comment:
   Add host & port to flights so that when the scheduler gets a flight request, 
it can go find the source. Architecturally, I see this as a good 
canonicalization of the source of Flight data, but I'm happy to discuss here!



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

Reply via email to