jarohen opened a new pull request, #40987:
URL: https://github.com/apache/arrow/pull/40987

   ### Rationale for this change
   
   Extension vectors are unsupported in `JsonFileWriter`/`JsonFileReader`. We 
use these classes to test our Arrow support, but when we try to round-trip data 
with (e.g.) UUIDs it fails.
   
   ### What changes are included in this PR?
   
   * `JsonFileWriter`: we delegate to the existing implementation for the 
underlying vector.
   * `JsonFileReader`: we register all of the registered extension types as 
sub-types to the Jackson reader.
   
   ### Are these changes tested?
   
   Yes
   
   ### Are there any user-facing changes?
   
   One: the addition of `ExtensionTypeRegistry.getExtensionNames()` so that the 
`JsonFileReader` can register them as sub-types with Jackson. Doesn't seem 
ideal to introduce a public method just to be called from `JsonFileReader` so 
(unless this is actually desirable, of course!), am open to other suggestions.


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