zeroshade commented on issue #34292: URL: https://github.com/apache/arrow/issues/34292#issuecomment-1454201790
Just adding an `UnmarshalJSON` for the extension builder would work for just the extension builder, but wouldn't work for the nested scenarios. It might make sense to just make the `Unmarshal` method public on the builders which would then allow customizing when combined with adding the functionality to have it properly construct a custom builder type to wrap the ExtensionBuilder. Since all nested scenarios and `FromJSON` interact by way of calling the `unmarshal` function of the interface, making that function public would allow it to be overridden by custom builders. -- 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]
