zero323 commented on pull request #34320: URL: https://github.com/apache/spark/pull/34320#issuecomment-999824952
In general it looks reasonable (it is common to want to reuse inferred schema for example) ‒ my only concern is that it is going to break user snapshot tests and possibly affect things like language-to-language migrations. So it ifs going to be merged it has to be included as potentially breaking change in the release notes. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
