MaxGekk edited a comment on pull request #31101: URL: https://github.com/apache/spark/pull/31101#issuecomment-757512206
> can you make a backporting PR for branch-3.1? It has a conflict. @HyukjinKwon Here are the backports: - `branch-3.0`: https://github.com/apache/spark/pull/31116 - `branch-3.1`: https://github.com/apache/spark/pull/31115 BTW, while backporting this PR to 3.1/3.0, I have realised that the test from this PR runs twice v1 In-Memory catalog. The PR https://github.com/apache/spark/pull/31117 runs the test for v1 Hive External catalog too. Just in case, I could make similar fix in v2 `ALTER TABLE .. ADD PARTITION` but I cannot write a test for that because existing V2 In-Memory catalog keeps partition data in memory, and partition locations are not taken into account. So, I cannot add new partition with data via SQL API. Is it ok if I make such fix without any test? cc @cloud-fan ---------------------------------------------------------------- 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]
