pitrou commented on code in PR #41257:
URL: https://github.com/apache/arrow/pull/41257#discussion_r1592665683


##########
docs/source/format/CanonicalExtensions.rst:
##########
@@ -51,7 +51,7 @@ types:
 
   3) Its serialization *must* be described in the proposal and should
      not require unduly implementation work or unusual software dependencies
-     (for example, a trivial custom text format or JSON would be acceptable).
+     (for example, a trivial custom text format or YAML would be acceptable).

Review Comment:
   This sentence is **not** an example of a canonical extension type. It 
describes how canonical extension types _may_ be serialized as metadata.
   
   So this should remain JSON. We would definitely not allow YAML as the 
serialization format for canonical extension types.
   



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