[GitHub] [arrow] pitrou commented on a change in pull request #9633: ARROW-11560: [C++][FlightRPC] fix mutex error on SIGINT

pitrou commented on a change in pull request #9633: URL: https://github.com/apache/arrow/pull/9633#discussion_r588145650 ## File path: cpp/src/arrow/flight/server.cc ## @@ -791,16 +799,30 @@ struct FlightServerBase::Impl { void DoHandleSignal(int signum) { got_signal_

[GitHub] [arrow] pitrou commented on a change in pull request #9633: ARROW-11560: [C++][FlightRPC] fix mutex error on SIGINT

pitrou commented on a change in pull request #9633: URL: https://github.com/apache/arrow/pull/9633#discussion_r588143037 ## File path: cpp/src/arrow/flight/server.cc ## @@ -892,6 +914,25 @@ Status FlightServerBase::Serve() { impl_->old_signal_handlers_.clear(); impl_->run

[GitHub] [arrow] pitrou commented on a change in pull request #9633: ARROW-11560: [C++][FlightRPC] fix mutex error on SIGINT

pitrou commented on a change in pull request #9633: URL: https://github.com/apache/arrow/pull/9633#discussion_r587752941 ## File path: cpp/src/arrow/flight/server.cc ## @@ -899,6 +921,13 @@ Status FlightServerBase::Serve() { ARROW_ASSIGN_OR_RAISE(old_handler, SetSignalHand