kou commented on code in PR #36009:
URL: https://github.com/apache/arrow/pull/36009#discussion_r1225045025
##########
format/Flight.proto:
##########
@@ -189,6 +191,31 @@ message Result {
bytes body = 1;
}
+/*
+ * The result of the CancelFlightInfo action.
+ *
+ * The result should be stored in Result.body.
+ */
+message ActionCancelFlightInfoResult {
+ enum CancelResult {
Review Comment:
We may want to remove `Cancel`:
```suggestion
enum Result {
```
Or we may want to move this to top-level and return this enum directly
instead of wrapping by `ActionCancelFlightInfoResult` message.
##########
format/Flight.proto:
##########
@@ -189,6 +191,31 @@ message Result {
bytes body = 1;
}
+/*
+ * The result of the CancelFlightInfo action.
+ *
+ * The result should be stored in Result.body.
+ */
+message ActionCancelFlightInfoResult {
Review Comment:
We may want to remove `Action`:
```suggestion
message CancelFlightInfoResult {
```
--
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]
