[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user MasterDDT commented on the issue: https://github.com/apache/spark/pull/14092 Closing this --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/14092 @MasterDDT I am sorry that I didn't get back to you sooner. We have recently merged PR https://github.com/apache/spark/pull/14841 which implements similar functionality. Cloud you close this PR?

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user MasterDDT commented on the issue: https://github.com/apache/spark/pull/14092 cc @hvanhovell --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14092 **[Test build #3169 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3169/consoleFull)** for PR 14092 at commit [`b4b02bf`](https://github.com/apache/spark/commit

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14092 **[Test build #3168 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3168/consoleFull)** for PR 14092 at commit [`b4b02bf`](https://github.com/apache/spark/commit

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14092 **[Test build #3169 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3169/consoleFull)** for PR 14092 at commit [`b4b02bf`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14092 **[Test build #3168 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3168/consoleFull)** for PR 14092 at commit [`b4b02bf`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/14092 Ok to test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user MasterDDT commented on the issue: https://github.com/apache/spark/pull/14092 cc @JoshRosen @rxin I wasn't sure if the right fix here is that `Expression` should override `equals` and use `semanticEquals`, that would be a bigger change but I think would work. Also

[GitHub] spark issue #14092: [SPARK-16419][SQL] EnsureRequirements adds extra Sort to...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14092 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feat