scovich commented on PR #7276: URL: https://github.com/apache/arrow-rs/pull/7276#issuecomment-2837012852
> @scovich and @Blizzara -- would this PR be superceded by this PR? > > * [Add custom decoder in arrow-json #7442](https://github.com/apache/arrow-rs/pull/7442) > Potentially? But I'd be very interested to see how one could actually achieve the same result with that other PR in practice. I suspect it would require a pretty annoying exercise to replicate each decoder type, just to inject the null handling. Maybe each custom decoder could be a thin wrapper around the default one... except I don't think any of the decoders are public today (see e.g. https://docs.rs/arrow-json/55.0.0/arrow_json/all.html)? -- 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]
