khwilson commented on PR #43849: URL: https://github.com/apache/arrow/pull/43849#issuecomment-2323129552
Apologies for my delay, @ianmcook . Ended up just taking a nap on that flight. :-) I've taken a stab at disambiguating "parameterization." To do so, I decided to explicitly call the metadata captured by the `ARROW:extension:metadata` field "parameters." If this is outside the bounds of a PR and should be discussed more broadly, please let me know. The only remaining unaddressed comment from your review about the UUID type in Columnar.rst. However, I actually don't read that one as needing to be replaced _unless_ the intention of _canonical_ extension types is that all Arrow implementations _must_ or _should_ support them. Reading the page on [canonical extensions](https://arrow.apache.org/docs/format/CanonicalExtensions.html) it is unclear to me if arrow implementations _must_ , _should_ , or _may_ support them. If it is _may_, then I think the text of Columnar is actually quite good, though I might modify it as follows: > For example, Arrow specifies a canonical extension type that represents a UUID as a ``FixedSizeBinary(16)``. Arrow implementations are not required to support canonical extensions, so an implementation that does not support this UUID type will simply interpret it as a ``FixedSizeBinary(16)`` and pass along the ``custom_metadata`` in subsequent Arrow protocol messages. -- 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]
