Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22715
@cloud-fan , sure, I'll submit a follow-up PR for this. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22715
Hi @mgaido91 , since you are the major author of this part, do you have
time to open a PR and move `outputOrdering` to the main constructor? Thanks in
advance!
---
-
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22715
sorry @gatorsmile that is fine, my bad. A late LGTM to this, despite
probably @cloud-fan 's comment make, we probably should have just put it as a
part of the main constructor...
---
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22715
Thank you, @gatorsmile and @mgaido91 .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22715
Merged to master/branch-2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22715
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22715
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97350/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22715
**[Test build #97350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97350/testReport)**
for PR 22715 at commit
[`cff78e9`](https://github.com/apache/spark/commit/c