xuanyuanking edited a comment on pull request #27627: URL: https://github.com/apache/spark/pull/27627#issuecomment-637271156
> How about we merge it to master only first, and wait for the schema incompatibility check to be done? Agree. > Just to avoid redundant efforts, have you look into #24173? If your approach is different than #24173, what approach you will be proposing? @HeartSaVioR Thanks for the reminding. I also looked into #24173 before. My approach is checking the underlying unsafe row format instead of adding a new schema file in the checkpoint. It is decided by the requirement of detecting the format changing during migration, which has no chance for the user to create a schema file. But I think our approaches can complement each other. Let's discuss in my newly created PR, I'll submit it late today. ---------------------------------------------------------------- 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]
