Skip to content

Commit

Permalink
Add application metadata field to FlightData message
Browse files Browse the repository at this point in the history
  • Loading branch information
lidavidm committed Jun 26, 2019
1 parent c631c9b commit 86f4789
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions format/Flight.proto
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ service FlightService {
* number. In the latter, the service might implement a 'seal' action that
* can be applied to a descriptor once all streams are uploaded.
*/
rpc DoPut(stream FlightData) returns (PutResult) {}
rpc DoPut(stream FlightData) returns (stream PutResult) {}

/*
* Flight services can support an arbitrary number of simple actions in
Expand Down Expand Up @@ -285,6 +285,11 @@ message FlightData {
*/
bytes data_header = 2;

/*
* Application-defined metadata.
*/
bytes data_app_metadata = 3;

/*
* The actual batch of Arrow data. Preferably handled with minimal-copies
* coming last in the definition to help with sidecar patterns (it is
Expand All @@ -295,7 +300,8 @@ message FlightData {
}

/**
* The response message (currently empty) associated with the submission of a
* DoPut.
* The response message associated with the submission of a DoPut.
*/
message PutResult {}
message PutResult {
bytes data_app_metadata = 1;
}

0 comments on commit 86f4789

Please sign in to comment.