Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/19559
Sure, I will submit it later.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
Could you submit a backport PR to 2.2?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83138/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83138/testReport)**
for PR 19559 at commit
[`a57fb81`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83138/testReport)**
for PR 19559 at commit
[`a57fb81`](https://github.com/apache/spark/commit/a5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
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/19559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83122/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83122 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83122/testReport)**
for PR 19559 at commit
[`2efd4f7`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83122/testReport)**
for PR 19559 at commit
[`2efd4f7`](https://github.com/apache/spark/commit/2e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83121/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83121 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83121/testReport)**
for PR 19559 at commit
[`846cee4`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83121 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83121/testReport)**
for PR 19559 at commit
[`846cee4`](https://github.com/apache/spark/commit/84
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83117/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83117 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83117/testReport)**
for PR 19559 at commit
[`5baf98e`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83117 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83117/testReport)**
for PR 19559 at commit
[`5baf98e`](https://github.com/apache/spark/commit/5b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83114/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83114 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83114/testReport)**
for PR 19559 at commit
[`87c2073`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83114 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83114/testReport)**
for PR 19559 at commit
[`87c2073`](https://github.com/apache/spark/commit/87
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83111/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83110/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
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/19559
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83111/testReport)**
for PR 19559 at commit
[`a96d945`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83110/testReport)**
for PR 19559 at commit
[`4b13343`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
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/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83112/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83112 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83112/testReport)**
for PR 19559 at commit
[`36b4bbb`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83112 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83112/testReport)**
for PR 19559 at commit
[`36b4bbb`](https://github.com/apache/spark/commit/36
Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/19559
Yes, ordering in `Sort(ordering, global, child)` is resolved in
`resolveExpression`
---
-
To unsubscribe, e-mail: reviews-unsu
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
`select a from t order by current_date` is not working?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
You do not need to close it and just need to resolve the conflicts. If you
are not familiar with it, I can wait and merge this PR first.
---
Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/19559
OK, I will close this PR after review and submit a new one, after merging
https://github.com/apache/spark/pull/19585
---
-
To
Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/19559
@gatorsmile
It seems that we should also support this logic in `resolveExpressions` for
Sort plan.
`select a from t order by current_date`
Therefore, I think current `resolveAsLiteralF
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
LGTM and cc @hvanhovell for a final sign-off.
BTW, I might merge https://github.com/apache/spark/pull/19585 before this
PR. Sorry for the conflicts.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83111/testReport)**
for PR 19559 at commit
[`a96d945`](https://github.com/apache/spark/commit/a9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83110/testReport)**
for PR 19559 at commit
[`4b13343`](https://github.com/apache/spark/commit/4b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83081/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83081 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83081/testReport)**
for PR 19559 at commit
[`b8075e1`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83080 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83080/testReport)**
for PR 19559 at commit
[`6c932b7`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
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/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83080/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83081 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83081/testReport)**
for PR 19559 at commit
[`b8075e1`](https://github.com/apache/spark/commit/b8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
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/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83078/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83078 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83078/testReport)**
for PR 19559 at commit
[`5323fbb`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83080 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83080/testReport)**
for PR 19559 at commit
[`6c932b7`](https://github.com/apache/spark/commit/6c
Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/19559
@gatorsmile @gatorsmile
There are still two issues need to be figured out.
(1)It will be complicated to determine whether a literal function should be
resolved as Expression or NamedExpres
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83078 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83078/testReport)**
for PR 19559 at commit
[`5323fbb`](https://github.com/apache/spark/commit/53
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83077/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83077 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83077/testReport)**
for PR 19559 at commit
[`5323fbb`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83077 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83077/testReport)**
for PR 19559 at commit
[`5323fbb`](https://github.com/apache/spark/commit/53
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83076/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83076/testReport)**
for PR 19559 at commit
[`2d42abf`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83076 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83076/testReport)**
for PR 19559 at commit
[`2d42abf`](https://github.com/apache/spark/commit/2d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
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/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83071/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83071 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83071/testReport)**
for PR 19559 at commit
[`d485e25`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83071 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83071/testReport)**
for PR 19559 at commit
[`d485e25`](https://github.com/apache/spark/commit/d4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19559
Merged build finished. Test FAILed.
---
-
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/19559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83044/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83044 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83044/testReport)**
for PR 19559 at commit
[`c38ab56`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83044 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83044/testReport)**
for PR 19559 at commit
[`c38ab56`](https://github.com/apache/spark/commit/c3
Github user DonnyZone commented on the issue:
https://github.com/apache/spark/pull/19559
@gatorsmile Thank for your advice, I will work on it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19559
@DonnyZone Analyzer is the best place to fix the issue.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
73 matches
Mail list logo
