[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191624811 @srowen . I made a quick fix. Could you review #11488 , please? --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191590972 Oops, I'll find a quick fix ASAP. --- 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

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191586179 Then, will Spark 2.0.0 support both Scala versions like now? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191586833 Yeah at the moment we need to make sure code still compiles for 2.10. I think this doesn't compile unfortunately: https://amplab.cs.berkeley.edu/jenkins/view/Spark%

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191585706 Thank you, @srowen . You mean the scope of this was Scala 2.11.7 and there is some more work remaining for Scala 2.10, right? I see. --- If your project is

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191582359 Yeah I just want to make sure that this all still compiled with Scala 2.10 for now. The test PR builder wouldnt catch it but other jobs would after. --- If your projec

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191578985 Thank you for merging! --- 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 h

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11479 --- 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 ena

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191577862 Merging in master. Thanks. --- 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 th

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191576684 Oh, thank you, @rxin . I missed your message here. --- 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-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191575961 **[Test build #2603 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2603/consoleFull)** for PR 11479 at commit [`5977253`](https://

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191536573 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

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191536808 **[Test build #2603 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2603/consoleFull)** for PR 11479 at commit [`5977253`](https://g

[GitHub] spark pull request: [SPARK-13627][SQL][YARN] Fix simple deprecatio...

2016-03-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11479#issuecomment-191525506 Hi, @srowen . Could you review this PR? This PR fixes 6 types of simple deprecations. --- If your project is set up for it, you can reply to this email and h