felipecrv commented on code in PR #43330:
URL: https://github.com/apache/arrow/pull/43330#discussion_r1685509653
##########
cpp/src/arrow/flight/types.h:
##########
@@ -1078,6 +1068,29 @@ struct ARROW_FLIGHT_EXPORT CloseSessionRequest
static arrow::Status Deserialize(std::string_view serialized,
CloseSessionRequest* out);
};
+/// \brief The result of closing a session.
+enum class CloseSessionStatus : int8_t {
+ // \brief The session close status is unknown.
+ //
+ // Servers should avoid using this value (send a NOT_FOUND error if the
requested
+ // session is not known). Clients can retry the request.
+ kUnspecified,
+ // \brief The session close request is complete.
+ //
+ // Subsequent requests with the same session produce a NOT_FOUND error.
+ kClosed,
+ // \brief The session close request is in progress.
+ //
+ // The client may retry the request.
+ kClosing,
+ // \brief The session is not closeable.
+ //
+ // The client should not retry the request.
+ kNotClosable
Review Comment:
Funny typo here. It's "not_closEable" in `Flight.proto`.
--
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]
