paleolimbot commented on PR #41823: URL: https://github.com/apache/arrow/pull/41823#issuecomment-2139580401
> otherwise this "arrow.other" type feels like a workaround for a limitation in the C++ API. I think they are considering two different scenarios: an "extension type" is perhaps something that has a definition or specification somewhere, whereas when the Postgres driver encounters a type that it didn't know about at compile time, all ADBC has is `const char* typname` and `const char* vendor_name` and it needs a way to pass that on. There is no guarantee regarding the content of the vendor name or the type name, so you might need something like `adbc:::PostgreSQL:::Some.Type` in the off chance one of the components contained a period. I would also like to see pyarrow flag unknown extension types, but using that system for this use-case feels very hack-y to me. -- 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
