Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22728
(From https://github.com/apache/spark/pull/22773#issuecomment-432917994)
@gatorsmile and @cloud-fan, let's say this will break `DESCRIBE FUNCTION
EXTENDED`. Should we update migration guide as
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22728
thanks for your work @cloud-fan !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22728
FYI, I tried both hive and presto, neither of them supports multi-column
count.
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22728
Merged to master and branch-2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97420/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22728
**[Test build #97420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97420/testReport)**
for PR 22728 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22728
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/22728
**[Test build #97420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97420/testReport)**
for PR 22728 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97400/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22728
**[Test build #97400 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97400/testReport)**
for PR 22728 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97401/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
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/22728
**[Test build #97401 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97401/testReport)**
for PR 22728 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22728
Yea, it is definitely good to add document and test for current behavior.
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22728
BTW MySQL doesn't support `count(a, b)` but supports `count(distinct a,
b)`, the result is same as Spark.
---
-
To
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22728
this is indeed the behavior I'd expect. Good to add tests to enforce the
behavior. Did you check other RDBMs apart from Postgres?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22728
**[Test build #97401 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97401/testReport)**
for PR 22728 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22728
**[Test build #97400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97400/testReport)**
for PR 22728 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22728
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22728
cc @gatorsmile @mgaido91 @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
27 matches
Mail list logo
