[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread cloud-fan
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] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread jinxing64
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread AmplabJenkins
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread AmplabJenkins
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] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread SparkQA
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread zsxwing
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread vanzin
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread vanzin
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread zsxwing
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

[GitHub] spark issue #18593: [SPARK-21369][Core]Don't use Scala Tuple2 in common/netw...

2017-07-10 Thread SparkQA
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