srowen commented on pull request #31461: URL: https://github.com/apache/spark/pull/31461#issuecomment-772886355
Ah, I get it. That's what I was looking for. `InternalRow` is not technically private, though informally so, being part of the `.catalyst` package. For those that have much more experience implementing a UDT with the hack: in practice must you assume it's an `InternalRow`? If so then yeah I see the argument that this API needs to change first before being made public. We _can_ just open it up to continue to enable the hack, and that buys something. But it makes it harder to modify the API to something more proper later. So what about #16478 @viirya ? is that still a sound proposal for refactoring the interface? That seems more ideal to add first. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
