[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12666 --- 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 feature is

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214699695 Merged to master --- 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

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214695045 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

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214695047 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214694869 **[Test build #56986 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56986/consoleFull)** for PR 12666 at commit

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214666557 **[Test build #56986 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56986/consoleFull)** for PR 12666 at commit

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-26 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214666328 Jenkins retest this please --- 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

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214613541 **[Test build #2883 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2883/consoleFull)** for PR 12666 at commit

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214611155 LGTM pending Jenkins. --- 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

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214611080 **[Test build #2883 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2883/consoleFull)** for PR 12666 at commit

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread sbcd90
Github user sbcd90 commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214586830 Hi @srowen , changed to `IllegalArgumentException`. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12666#discussion_r61005340 --- Diff: core/src/main/scala/org/apache/spark/scheduler/Pool.scala --- @@ -55,6 +55,8 @@ private[spark] class Pool( new

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread sbcd90
Github user sbcd90 commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214548968 Hi @jaceklaskowski , Thanks for your review. I think it is safe that we make the changes at both the places. I also checked for the occurrence of scheduler

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread jaceklaskowski
Github user jaceklaskowski commented on a diff in the pull request: https://github.com/apache/spark/pull/12666#discussion_r60991107 --- Diff: core/src/main/scala/org/apache/spark/scheduler/Pool.scala --- @@ -55,6 +55,8 @@ private[spark] class Pool( new

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12666#issuecomment-214484848 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: [SPARK-14889][Spark Core] scala.MatchError: NO...

2016-04-25 Thread sbcd90
GitHub user sbcd90 opened a pull request: https://github.com/apache/spark/pull/12666 [SPARK-14889][Spark Core] scala.MatchError: NONE (of class scala.Enumeration) when spark.scheduler.mode=NONE ## What changes were proposed in this pull request? Handling exception for the