amaliujia commented on PR #41426: URL: https://github.com/apache/spark/pull/41426#issuecomment-1581895934
> BTW, sorry for asking this question after merging. > > 1. What is the minimal target for migration in order to help `connect` module? According to the PR description, `Row` is also mentioned. So,`sql/catalyst/src/main/scala/org/apache/spark/sql/types` is not enough? > 2. Does it cause many MIMA exceptions against old `catalyst` module? In current scope, Row, coders, data types are required. However we may discover more during the process. -- 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]
