ianmcook commented on code in PR #47456:
URL: https://github.com/apache/arrow/pull/47456#discussion_r2319729955


##########
docs/source/format/CanonicalExtensions.rst:
##########
@@ -45,7 +45,11 @@ types:
 
 * The specification text to be added *must* follow these requirements:
 
-  1) It *must* define a well-defined extension name starting with "``arrow.``".
+  1) It *must* define a well-defined extension name starting with an allowed 
prefix.
+     The currently allowed prefixes are:
+     * "``arrow.``" - For general-purpose canonical extension types.
+     * "``parquet.``" - For canonical extension types that are intended 
primarily for

Review Comment:
   In a discussion about this in the Parquet sync meeting today, there was a 
general preference for `arrow.parquet.` because it does not expand the 
previously reserved namespaces and avoids possible name collisions. It sounds 
like we'll go with that unless there are well-reasoned objections.



-- 
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]

Reply via email to