Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23075
@ssimeonov Thank you for your contribution to Spark. We wait for your next
PR.
---
-
To unsubscribe, e-mail:
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/23075
Also backported to 2.3/2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/23075
Merging to master. Thanks!
---
-
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/23075
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/23075
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99061/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23075
**[Test build #99061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99061/testReport)**
for PR 23075 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23075
**[Test build #99061 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99061/testReport)**
for PR 23075 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23075
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/23075
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/23075
Let's see if this works :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/23075
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23075
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/23075
Ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23075
@hvanhovell Could you trigger a build, please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23075
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ssimeonov commented on the issue:
https://github.com/apache/spark/pull/23075
@MaxGekk done
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23075
> Which suite would you like me to add the test in?
Please, create new test suite.
---
-
To unsubscribe, e-mail:
Github user ssimeonov commented on the issue:
https://github.com/apache/spark/pull/23075
@MaxGekk I see no mention of `AggregateExpression` in the
`sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions`
subtree. Which suite would you like me to add the test in?
---
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23075
Would you mind to add a test for case which you described in JIRA ticket?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23075
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23075
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23075
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
22 matches
Mail list logo