peter-toth commented on PR #59599: URL: https://github.com/apache/spark/pull/59599#issuecomment-6470353909
Thanks, @ammarchalifah. Findings 27 and 29-31 are resolved at b2ae907. The 26 tests pass, 17 of them fail on branch-4.1, and the new ones fail without their fix. For 28, the description now starts with the new title, but the PR title itself is unchanged. Could you set the title to `[SPARK-60232][SPARK-60552][SQL][4.1] ...` and drop that first line from the description? Please keep the other tickets out of this PR. It already fixes two, and each of them is a different mechanism: - SPARK-59121 and SPARK-58996 are fixed on 4.2 and later, so they need backports of their own. - SPARK-60460 and SPARK-60579 are 4.1/4.0-only bugs, each with its own fix. You're welcome to take any of them in a separate PR. That also keeps the reviews and a later 4.0 backport simpler. -- 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]
