alinaliBQ opened a new issue, #47855:
URL: https://github.com/apache/arrow/issues/47855
### Describe the enhancement requested
This issue is an optional one to consider.
Currently, `GetSqlInfo` doesn't have any additional properties. One can
create custom properties in `GetSqlInfo` so that the ODBC can use these
properties directly. The goals is to have 1 property per `case` statement For
example, instead of:
```
case SqlInfoOptions::FLIGHT_SQL_SERVER_NAME: {
std::string server_name(
reinterpret_cast<StringScalar*>(scalar->child_value().get())->view());
info_[SQL_SERVER_NAME] = server_name;
info_[SQL_DBMS_NAME] = server_name;
info_[SQL_DATABASE_NAME] = server_name;
break;
}
```
we have:
```
case SqlInfoOptions::FLIGHT_SQL_SERVER_NAME: {
info_[SQL_SERVER_NAME] = server_name;
break;
}
case SQL_DBMS_NAME: {
info_[SQL_DBMS_NAME] = dbms_name;
break;
}
case SQL_DATABASE_NAME: {
info_[SQL_DATABASE_NAME] = database_name;
break;
}
```
`GetSqlInfo` current definition:
https://github.com/apache/arrow/blob/bab558061696ddc1841148d6210424b12923d48e/cpp/src/arrow/flight/sql/client.cc#L558-L564
### Component(s)
FlightRPC, C++
--
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]
