lidavidm commented on PR #34817: URL: https://github.com/apache/arrow/pull/34817#issuecomment-1815393103
> Thoughts on where to put this—Proto defs maybe? We can put them in the docs in this case, since Protobuf doesn't really have a way to do constants. https://github.com/apache/arrow/blob/main/docs/source/format/FlightSql.rst > Related, (a) we were also discussing maybe adding (later) a verb to list available option keys, and (b) possibly (I think maybe not—your opinion?) documenting the recommended practice of setting options before other (e.g. getFI) calls and all-at-once, but I think that's pretty implementation-dependant and probably doesn't belong in Arrow unlike the suggested option key conventions... (a) is a possible extension, and I think one that would be useful. (b) could be something that goes into the docs as something that is suggested but not part of the protocol itself. -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
