zeroshade commented on code in PR #40284:
URL: https://github.com/apache/arrow/pull/40284#discussion_r1509351090
##########
cpp/src/arrow/flight/sql/server_session_middleware.cc:
##########
@@ -80,7 +80,7 @@ class ServerSessionMiddlewareImpl : public
ServerSessionMiddleware {
Status CloseSession() override {
const std::lock_guard<std::shared_mutex> l(mutex_);
- if (static_cast<bool>(session_)) {
+ if (!static_cast<bool>(session_)) {
Review Comment:
@joellubi Can you add a corresponding fix to the client middleware that
matches #40071 which @indigophox mentioned? Or at minimum, file a follow-up
issue for this so we don't forget it
--
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]
