viirya commented on pull request #31461: URL: https://github.com/apache/spark/pull/31461#issuecomment-773084514
> So what about #16478 @viirya ? is that still a sound proposal for refactoring the interface? That seems more ideal to add first. Hm, yea, the refactoring is trying to hide internal classes such as `UnsafeArrayData`, `GenericInternalRow`, etc. from UDT usage. So the developers use Scala classes when writing UDTs. Although as UDT is private for many years, I guess the developers just use Spark namespace to code their UDTs with internal classes for a while. Technically, yes, it seems more ideal to refactor and hide internal classes 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]
