indigophox commented on code in PR #34817:
URL: https://github.com/apache/arrow/pull/34817#discussion_r1170322012
##########
cpp/src/arrow/flight/sql/server.cc:
##########
@@ -1072,6 +1260,18 @@ Status FlightSqlServerBase::EndTransaction(const
ServerCallContext& context,
return Status::NotImplemented("EndTransaction not implemented");
}
+arrow::Result<ActionSetSessionOptionsResult>
FlightSqlServerBase::SetSessionOptions (
+ const ServerCallContext& context,
+ const ActionSetSessionOptionsRequest& request) {
+ return Status::NotImplemented("SetSessionOptions not implemented");
+}
Review Comment:
* Handler would request access to the session object from server middleware.
* Middleware would either:
* Already have found a valid session id cookie on call entry and set the
session object for that call-local middleware instance
* Or, if the session id cookie was not provided by the client the server
session middleware would generate a new session object and corresponding ID
* Middleware would then set the response Set-Cookie header accordingly, and
return the new OR looked-up session object to the handler.
--
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]
