indigophox commented on code in PR #34817:
URL: https://github.com/apache/arrow/pull/34817#discussion_r1156269692


##########
cpp/src/arrow/flight/sql/server.h:
##########
@@ -225,6 +225,27 @@ struct ARROW_FLIGHT_SQL_EXPORT 
ActionCreatePreparedStatementResult {
   std::string prepared_statement_handle;
 };
 
+/// \brief A request to close the open client session.
+struct ARROW_FLIGHT_SQL_EXPORT ActionCloseSessionRequest {};

Review Comment:
   [see 
[above](https://github.com/apache/arrow/pull/34817#pullrequestreview-1366808831)
 for discussion]
   
   More specifically, the SetSessionOptions implementation would inject the 
Set-Cookie response header as appropriate.  Or in the alternate case of an 
existing session, it would be accessed via the request headers.



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