Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #97798 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97798/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97731/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97710/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #97731 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97731/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #97710 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97710/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #97731 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97731/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #97710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97710/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/22047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96995/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
**[Test build #96995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96995/testReport)**
for PR 22047 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
@cloud-fan please see my comment
[link](https://github.com/apache/spark/pull/22047#issuecomment-411536039). I
had tried to rewrite using max and min as suggested by Herman and Reynold in
the
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22047
is it possible to rewrite these 3 new functions with existing expression?
e.g.
`every(col)` -> `count(if (col) null else 1) == 0`
`any(col)` -> `count(if (col) 1 else null) > 0`
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
**[Test build #96995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96995/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96978/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #96978 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96978/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #96978 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96978/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
retest this please
---
-
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/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96968/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #96968 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96968/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/22047
**[Test build #96968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96968/testReport)**
for PR 22047 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
@gatorsmile First of all, thank you very much . Actually the added
aggregates weren't null filtering. I have fixed the issue and have added
additional test cases. Thank you.
---
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
@gatorsmile Thanks.. I will check.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22047
Let me post something I wrote recently. Could you add test cases to ensure
that we do not break the "Ignore NULLs" policy
> All the set/aggregate functions ignore NULLs. The typical
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96872/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
**[Test build #96872 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96872/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #96872 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96872/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/22047
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/22047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94602/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94602 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94602/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94602 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94602/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
retest this please
---
-
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/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94588/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94588 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94588/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94588 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94588/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94586 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94586/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94586/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94586 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94586/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22047
> Please give credit to @ptkool for this work.
FWIW, we can now credit to multiple people per
https://github.com/apache/spark/commit/51bee7aca13451167fa3e701fcd60f023eae5e61
:-)
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94459/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94459/testReport)**
for PR 22047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94459 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94459/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/22047
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94456/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94456 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94456/testReport)**
for PR 22047 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
@gatorsmile I tried to implement the rewrites suggested in the original PR.
It does not seem very straightforward to me. The basic issue is, we are unable
to replace the aggregate expression
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94456 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94456/testReport)**
for PR 22047 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
76 matches
Mail list logo
