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


##########
format/FlightSql.proto:
##########
@@ -1842,6 +1842,94 @@ message ActionCancelQueryResult {
   CancelResult result = 1;
 }
 
+/*
+ * Request message for the "Close Session" action.
+ */
+message ActionCloseSessionRequest {

Review Comment:
   ServerSessionMiddleware (builds but I am still getting tests sorted) has 
been added to the PR as discussed.  Please let me know if you have any thoughts 
on ways you'd prefer the mechanics to be different.  (ServerSessionMiddleware 
also needs a bit more documentation possibly, again comments welcome but I will 
be reviewing that anyways.)



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