Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20254
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86211/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20254
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20254
**[Test build #86211 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86211/testReport)**
for PR 20254 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20254
@henryr Since Spark 2.3, Spark SQL documents all the behavior changes in
[Migration
Guides](https://spark.apache.org/docs/latest/sql-programming-guide.html#migration-guide).
Hopefully, this can
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20254
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20254
**[Test build #86211 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86211/testReport)**
for PR 20254 at commit
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/20254
Thanks all for the pointers and feedback! I've removed the references to
the behavior before 2.0, and now the changes just make it explicit that this is
`EXCEPT DISTINCT` (I appreciate that that's
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20254
Just FYI, in ANSI SQL, `EXCEPT` = `EXCEPT DISTINCT`
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20254
@henryr could you update this PR to only include EXCEPT DISTINCT without
the notes
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20254
Yeah, we should document the behavior changes, but that was just a bug fix
for 100% following the semantics of ANSI-SQL `EXCEPT DISTINCT`
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20254
I see, from reading that PR I think perhaps we should reference migration
guide in sql programming guide instead of putting the whole description here.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20254
Also, cc @gatorsmile .
---
-
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/20254
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86068/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20254
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20254
**[Test build #86068 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86068/testReport)**
for PR 20254 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20254
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86067/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20254
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20254
**[Test build #86067 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86067/testReport)**
for PR 20254 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20254
**[Test build #86068 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86068/testReport)**
for PR 20254 at commit
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/20254
Done, thanks for pointing that out!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20254
**[Test build #86067 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86067/testReport)**
for PR 20254 at commit
21 matches
Mail list logo