scovich commented on PR #8998: URL: https://github.com/apache/arrow-rs/pull/8998#issuecomment-3661799335
> Big picture, what are your thoughts on the alternative approach that @alamb proposed above of adding the ability to register custom decoders on the reader and then exposing this an optional decoder that could be registered? This was essentially already discussed in #7442 and at the time the main concern seemed to be around making `Tape` public. TBH, I don't think I understand it well enough to have a strong opinion. In particular, I didn't follow how the proposed extension stuff would help solve what problem this PR has? There's a bunch of issues and pull requests around customizable JSON parsing, especially when it comes to error handling. But in theory variant shouldn't have that problem because every valid JSON value can be represented as some variant subtype. Some pull requests of my own also got caught up (and died) due to interactions with tape decoder, so my immediate reaction is to get something working here and worry about expanding the overall API as a separate effort. But again, low confidence answer. -- 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]
