westonpace commented on PR #13501: URL: https://github.com/apache/arrow/pull/13501#issuecomment-1235891475
That seems reasonable. Feel free to merge. You're basically working on the physical type and saving off the extension type to reattach later? I think, longer term, we might want to consider doing this at the plan level. So nodes never have to know about the existence of extension types, and we simply attach them, as needed, in the sink. Might want to wait until we have a few compute functions for extension types first though so there is a bit more example. -- 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]
