ennuite opened a new issue, #51114: URL: https://github.com/apache/arrow/issues/51114
### Describe the enhancement requested PR #49498 added `is_update` as `optional bool` (field 4) to the `ActionCreatePreparedStatementResult` protobuf message in `format/FlightSql.proto`. However, the C++ high-level wrapper layer has not been updated to expose this field. The following changes are needed in the core API (independent of any driver): 1. Add `std::optional<bool> is_update` to the `ActionCreatePreparedStatementResult` struct in `cpp/src/arrow/flight/sql/server.h` 2. Update `PackActionResult(ActionCreatePreparedStatementResult)` in `server.cc` to serialize the field to the protobuf message 3. Update `PreparedStatement::ParseResponse()` in `client.cc` to read `is_update` from the protobuf message 4. Add an `is_update()` accessor to the `PreparedStatement` class in `client.h`/`client.cc` This allows C++ Flight SQL server implementors to set `is_update` when creating a prepared statement, and C++ clients to read it — without requiring any driver changes. ### Component(s) FlightRPC Related - #49497 - #49498 - ODBC driver follow-up: #TODO -- 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]
