Thanks all for the comments. I see Tom also put up a PR to add this to DuckDB [1].
Do we need a vote for this? If so unless there are further comments I think we can start one. [1]: https://github.com/duckdb/duckdb/pull/3628 On Tue, May 10, 2022, at 13:31, David Li wrote: > For discussion I've put up https://github.com/apache/arrow/pull/13115 > to add this for the C data/stream interfaces. > > On Mon, May 9, 2022, at 15:42, Antoine Pitrou wrote: >> Le 09/05/2022 à 20:28, Tomek Drabas a écrit : >>> I am new to this board so please, let me know if any of this doesn't make >>> sense. >>> >>> I am building a FligthSQL example with DuckDB backend. DuckDB already has >>> an Arrow interface defined in duckdb.h that returns ArrowArray. However, >>> the import is not guarded in any way, and ArrowArray is redefined in >>> duckdb.h, so including arrow/c/bridge.h throws an error that ArrowArray is >>> defined in multiple places. >>> >>> I'd like to propose adding canonical guardrails in arrow/c/bridge.h to >>> avoid this. Is this the best way to do this? >> >> It should probably be included in the spec: >> https://arrow.apache.org/docs/format/CDataInterface.html#structure-definitions >> >> Regards >> >> Antoine. >> >> >>> >>> Thanks, >>> -Tom >>>
