[GitHub] [arrow-rs] alamb commented on pull request #4250: feat: update flight-sql to latest specs

["alamb (via GitHub)" <gi...@apache.org>]

alamb commented on PR #4250:
URL: https://github.com/apache/arrow-rs/pull/4250#issuecomment-1557210650

   >  @alamb - playing around with implementing a flight sql server I read a bit through the iOx implementation. Over there I found that the structs around sql infos (SqlInfoValue, SqlInfoList, SqlInfoUnionBuilder) are quite useful and probably non trivial to figure out for users. Do you think this is something that could be hoisted upstream?
   
   Yes, absolutely we could hoist this upstream. 
   
   I filed these tickets to do so:
   * `CommandGetSqlInfo`: https://github.com/apache/arrow-rs/issues/4256
   * `CommandGetXdbcTypeInfo`: https://github.com/apache/arrow-rs/issues/4257
   
   There are several other things that could be hoisted upstream (like the basic implementations for `GetTables` and `GetSchemas` both of which need basic filtering). Do you think those are worth pulling upstream too?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org