Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18593
LGTM, merging to master/2.2!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/18593
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fea
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18593
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18593
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79479/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18593
**[Test build #79479 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79479/testReport)**
for PR 18593 at commit
[`3d1a6ff`](https://github.com/apache/spark/commit/3
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/18593
@vanzin it doesn't work. I added scala-library into the test scope and
changed `org.apache.spark:spark-tags` back to the compile scope, but the build
didn't fail.
---
If your project is set up for
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18593
Also, an alternative would be to explicitly add the scala-library
dependency in test scope, in case some future dependency re-adds it in compile
scope.
---
If your project is set up for it, you can
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18593
LGTM.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the featu
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/18593
cc @cloud-fan @jinxing64
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18593
**[Test build #79479 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79479/testReport)**
for PR 18593 at commit
[`3d1a6ff`](https://github.com/apache/spark/commit/3d
10 matches
Mail list logo