Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93956/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21951
**[Test build #93956 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93956/testReport)**
for PR 21951 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21951
https://issues.apache.org/jira/browse/SPARK-24996 is created,
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21951
**[Test build #93956 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93956/testReport)**
for PR 21951 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21951
Thanks! Merged to master.
Please ignore the last commit.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21951
LGTM. shall we create a JIRA ticket to apply this to other
`DeclarativeAggregate`s?
---
-
To unsubscribe, e-mail:
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/21951
LGTM as well. Thanks a lot!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21951
LGTM.
On Thu, Aug 2, 2018 at 1:14 AM Xiao Li wrote:
> This will simplify the code and improve the readability. We can do the
> same in the other expression.
>
> â
>
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21951
This will simplify the code and improve the readability. We can do the same
in the other expression.
---
-
To unsubscribe,
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21951
Why would we want to use the DSL here? Do we use it in other expressions?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93918/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
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/21951
**[Test build #93918 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93918/testReport)**
for PR 21951 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21951
**[Test build #93918 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93918/testReport)**
for PR 21951 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21951
cc @hvanhovell @rednaxelafx @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21951
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/21951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user holdensmagicalunicorn commented on the issue:
https://github.com/apache/spark/pull/21951
@gatorsmile, thanks! I am a bot who has found some folks who might be able
to help with the review:@rxin, @cloud-fan and @ueshin
---
21 matches
Mail list logo