lidavidm commented on PR #34817:
URL: https://github.com/apache/arrow/pull/34817#issuecomment-1814534634
There's some CI failures as well:
```
/arrow/cpp/src/arrow/flight/sql/server_session_middleware.cc: In member
function 'std::pair<std::__cxx11::basic_string<char>,
std::shared_ptr<arrow::flight::sql::FlightSqlSession> >
arrow::flight::sql::ServerSessionMiddlewareFactory::GetNewSession()':
/arrow/cpp/src/arrow/flight/sql/server_session_middleware.cc:170:14: error:
'unique_lock' in namespace 'std' does not name a template type
170 | const std::unique_lock<std::shared_mutex> l(session_store_lock_);
| ^~~~~~~~~~~
/arrow/cpp/src/arrow/flight/sql/server_session_middleware.cc:19:1: note:
'std::unique_lock' is defined in header '<mutex>'; did you forget to '#include
<mutex>'?
```
```
/arrow/cpp/src/arrow/flight/sql/server_session_middleware.cc:35:
Single-parameter constructors should be marked explicit. [runtime/explicit] [5]
```
--
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]
