[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/12634 [SPARK-14869][SQL] Don't mask exceptions in ResolveRelations ## What changes were proposed in this pull request? In order to support SPARK-11197 (run SQL directly on files), we added some code in

[GitHub] spark pull request: [SPARK-14433][PySpark][ML]:PySpark ml Gaussian...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12402#issuecomment-213677565 **[Test build #56784 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56784/consoleFull)** for PR 12402 at commit [`16d94f4`](https://g

[GitHub] spark pull request: [SPARK-14433][PySpark][ML]:PySpark ml Gaussian...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12402#issuecomment-213677716 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213677747 cc @davies / @andrewor14 --- 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 thi

[GitHub] spark pull request: [SPARK-14433][PySpark][ML]:PySpark ml Gaussian...

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

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213678085 **[Test build #56787 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56787/consoleFull)** for PR 12634 at commit [`43cc738`](https://gi

[GitHub] spark pull request: [SPARK-14757][SQL] Corrected doGenCode() in Eq...

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12628#issuecomment-213678404 can you add a test case? --- 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 thi

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213678447 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 a

[GitHub] spark pull request: [SPARK-14857] [SQL] Table/Database Name Valida...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12618#issuecomment-213678602 **[Test build #56781 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56781/consoleFull)** for PR 12618 at commit [`bfe536e`](https://g

[GitHub] spark pull request: [SPARK-14857] [SQL] Table/Database Name Valida...

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

[GitHub] spark pull request: [SPARK-14857] [SQL] Table/Database Name Valida...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12618#issuecomment-213678648 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14867][BUILD] Make `build/mvn` to use t...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213678652 **[Test build #56780 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56780/consoleFull)** for PR 12631 at commit [`56355fa`](https://g

[GitHub] spark pull request: [SPARK-14867][BUILD] Make `build/mvn` to use t...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213678704 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14867][BUILD] Make `build/mvn` to use t...

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

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213679029 Merged build finished. Test FAILed. --- 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 projec

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213679008 **[Test build #56787 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56787/consoleFull)** for PR 12634 at commit [`43cc738`](https://g

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213679032 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: Investigate flaky test

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12565#issuecomment-213679503 **[Test build #2860 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2860/consoleFull)** for PR 12565 at commit [`a114466`](https://g

[GitHub] spark pull request: Investigate flaky test

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12565#issuecomment-213679584 **[Test build #2861 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2861/consoleFull)** for PR 12565 at commit [`a114466`](https://g

[GitHub] spark pull request: Investigate flaky test

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12565#issuecomment-213679617 **[Test build #2862 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2862/consoleFull)** for PR 12565 at commit [`a114466`](https://g

[GitHub] spark pull request: [SPARK-12148][SPARKR] SparkR: rename DataFrame...

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12621#issuecomment-213679816 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 t

[GitHub] spark pull request: [SPARK-12148][SPARKR] SparkR: rename DataFrame...

2016-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12621 --- 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: Investigate flaky test

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12565#issuecomment-213680468 **[Test build #56788 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56788/consoleFull)** for PR 12565 at commit [`a114466`](https://gi

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213681560 **[Test build #56790 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56790/consoleFull)** for PR 12560 at commit [`7b58c5c`](https://gi

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213681559 **[Test build #56789 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56789/consoleFull)** for PR 12634 at commit [`a1980dc`](https://gi

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213681615 **[Test build #56782 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56782/consoleFull)** for PR 12568 at commit [`3690c7c`](https://g

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213681727 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213681724 Merged build finished. Test FAILed. --- 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 projec

[GitHub] spark pull request: [SPARK-14863][SQL] Cache TreeNode's hashCode b...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12626#discussion_r60824742 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -71,7 +71,9 @@ object CurrentOrigin { } }

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213683704 Merged build finished. Test FAILed. --- 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 projec

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213683674 **[Test build #56790 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56790/consoleFull)** for PR 12560 at commit [`7b58c5c`](https://g

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213683707 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14441] [SQL] Consolidate DDL tests

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12347#issuecomment-213683729 Yea the current tests are pretty scattered and they are way too big to merge into a single one. Can you just close this pull request? Thanks. --- If your project is set

[GitHub] spark pull request: [SPARK-14871][SQL] Disable StatsReportListener...

2016-04-23 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/12635 [SPARK-14871][SQL] Disable StatsReportListener to declutter output ## What changes were proposed in this pull request? Spark SQL inherited from Shark to use the StatsReportListener. Unfortunately

[GitHub] spark pull request: [SPARK-14871][SQL] Disable StatsReportListener...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12635#issuecomment-213686652 **[Test build #56791 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56791/consoleFull)** for PR 12635 at commit [`18a7097`](https://gi

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213688248 **[Test build #56792 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56792/consoleFull)** for PR 12560 at commit [`ccd6ca0`](https://gi

[GitHub] spark pull request: [SPARK-14872][SQL] Restructure command package

2016-04-23 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/12636 [SPARK-14872][SQL] Restructure command package ## What changes were proposed in this pull request? This patch restructures sql.execution.command package to break the commands into multiple files,

[GitHub] spark pull request: [SPARK-14872][SQL] Restructure command package

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12636#issuecomment-213688570 cc @andrewor14 sorry for creating a lot of conflicts with your patch. I promise I will stop moving stuff around after this patch! --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-14872][SQL] Restructure command package

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12636#issuecomment-213688960 **[Test build #56793 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56793/consoleFull)** for PR 12636 at commit [`9bbe693`](https://gi

[GitHub] spark pull request: [SPARK-14871][SQL] Disable StatsReportListener...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12635#issuecomment-213689297 **[Test build #56791 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56791/consoleFull)** for PR 12635 at commit [`18a7097`](https://g

[GitHub] spark pull request: [SPARK-14871][SQL] Disable StatsReportListener...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12635#issuecomment-213689311 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14871][SQL] Disable StatsReportListener...

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

[GitHub] spark pull request: [SPARK-14868][BUILD] Enable NewLineAtEofChecke...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12632#issuecomment-213690810 **[Test build #56783 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56783/consoleFull)** for PR 12632 at commit [`d20e167`](https://g

[GitHub] spark pull request: [SPARK-14868][BUILD] Enable NewLineAtEofChecke...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12632#issuecomment-213690843 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12633#issuecomment-213690865 **[Test build #56785 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56785/consoleFull)** for PR 12633 at commit [`2d4ddc8`](https://g

[GitHub] spark pull request: [SPARK-14868][BUILD] Enable NewLineAtEofChecke...

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

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12633#issuecomment-213690917 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

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

[GitHub] spark pull request: Investigate flaky test

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213692318 **[Test build #56794 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56794/consoleFull)** for PR 12568 at commit [`ca46f4c`](https://gi

[GitHub] spark pull request: Investigate flaky test

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

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12633#issuecomment-213692659 **[Test build #56786 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56786/consoleFull)** for PR 12633 at commit [`9892ab5`](https://g

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12633#issuecomment-213692704 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

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

[GitHub] spark pull request: Investigate flaky test

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12565#issuecomment-213693621 **[Test build #56788 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56788/consoleFull)** for PR 12565 at commit [`a114466`](https://g

[GitHub] spark pull request: Investigate flaky test

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

[GitHub] spark pull request: Investigate flaky test

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12565#issuecomment-213693705 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: Investigate flaky test

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

[GitHub] spark pull request: [SPARK-13857][ML][WIP] Add "recommend all" fun...

2016-04-23 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/12574#discussion_r60825392 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/recommendation/MatrixFactorizationModel.scala --- @@ -261,58 +261,93 @@ object MatrixFactorizationModel

[GitHub] spark pull request: [SPARK-13857][ML][WIP] Add "recommend all" fun...

2016-04-23 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/12574#discussion_r60825409 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/recommendation/MatrixFactorizationModel.scala --- @@ -261,58 +261,93 @@ object MatrixFactorizationModel

[GitHub] spark pull request: [SPARK-13857][ML][WIP] Add "recommend all" fun...

2016-04-23 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/12574#discussion_r60825440 --- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala --- @@ -218,11 +292,135 @@ class ALSModel private[ml] ( predict(userF

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213694843 **[Test build #56789 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56789/consoleFull)** for PR 12634 at commit [`a1980dc`](https://g

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213694868 Merged build finished. Test FAILed. --- 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 projec

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213694869 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213696139 **[Test build #56795 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56795/consoleFull)** for PR 12416 at commit [`714afac`](https://gi

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213698709 **[Test build #56796 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56796/consoleFull)** for PR 12416 at commit [`1da5e91`](https://gi

[GitHub] spark pull request: [SPARK-14867][BUILD] Make `build/mvn` to use t...

2016-04-23 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213699239 Yes, on a system with Maven < 3.3.9 installed, you always have to pass `--force`. This would at least make it optional on the second or subsequent run. Ideally

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213700473 **[Test build #56795 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56795/consoleFull)** for PR 12416 at commit [`714afac`](https://g

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213700491 Build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213700495 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14872][SQL] Restructure command package

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12636#issuecomment-213703028 **[Test build #56793 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56793/consoleFull)** for PR 12636 at commit [`9bbe693`](https://g

[GitHub] spark pull request: [SPARK-14872][SQL] Restructure command package

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12636#issuecomment-213703073 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14872][SQL] Restructure command package

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213704837 **[Test build #56792 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56792/consoleFull)** for PR 12560 at commit [`ccd6ca0`](https://g

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-213704898 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/12637 [SPARK-14873] [CORE] Java sampleByKey methods take ju.Map but with Scala Double values; results in type Object ## What changes were proposed in this pull request? Java `sampleByKey` methods

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12637#discussion_r60826170 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java --- @@ -37,22 +37,19 @@ public static void main(String[]

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12637#issuecomment-213706212 **[Test build #56797 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56797/consoleFull)** for PR 12637 at commit [`0fe43b4`](https://gi

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12634#discussion_r60826409 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala --- @@ -32,6 +32,8 @@ trait CatalystConf { def optimizerInSet

[GitHub] spark pull request: [SPARK-14869][SQL] Don't mask exceptions in Re...

2016-04-23 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12634#issuecomment-213707877 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 enable

[GitHub] spark pull request: [SPARK-14863][SQL] Cache TreeNode's hashCode b...

2016-04-23 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12626#discussion_r60826489 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -71,7 +71,9 @@ object CurrentOrigin { } }

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213712873 **[Test build #56794 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56794/consoleFull)** for PR 12568 at commit [`ca46f4c`](https://g

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213713219 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-1989] [CORE] Exit executors faster if t...

2016-04-23 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12571#discussion_r60826861 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/SparkDeploySchedulerBackend.scala --- @@ -66,12 +66,20 @@ private[spark] class SparkDeploySc

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213723024 **[Test build #56796 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56796/consoleFull)** for PR 12416 at commit [`1da5e91`](https://g

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213723361 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

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

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12416#discussion_r60827197 --- Diff: mllib-local/src/main/scala/org/apache/spark/ml/linalg/Matrices.scala --- @@ -154,11 +172,12 @@ sealed trait Matrix extends Serializable { * @p

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213725055 This looks correct to my eyes. The Since annotation is in place as a non-test class; the existing test classes are moved into the test source root. The parent pom now de

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12637#issuecomment-213725948 Merged build finished. Test FAILed. --- 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 projec

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12637#issuecomment-213725950 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12637#issuecomment-213725832 **[Test build #56797 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56797/consoleFull)** for PR 12637 at commit [`0fe43b4`](https://g

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12637#issuecomment-213726045 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-14874][SQL][Streaming] Cleanup the usel...

2016-04-23 Thread lw-lin
GitHub user lw-lin opened a pull request: https://github.com/apache/spark/pull/12638 [SPARK-14874][SQL][Streaming] Cleanup the useless Batch class ## What changes were proposed in this pull request? The `Batch` class, which had been used to indicate progress in a stream, wa

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Cleanup the usel...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213726229 **[Test build #56798 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56798/consoleFull)** for PR 12638 at commit [`c79cba9`](https://gi

[GitHub] spark pull request: [SPARK-14873] [CORE] Java sampleByKey methods ...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12637#issuecomment-213726228 **[Test build #56799 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56799/consoleFull)** for PR 12637 at commit [`0fe43b4`](https://gi

[GitHub] spark pull request: [SPARK-14613][ML] Add @Since into the matrix a...

2016-04-23 Thread pravingadakh
Github user pravingadakh commented on a diff in the pull request: https://github.com/apache/spark/pull/12416#discussion_r60827833 --- Diff: mllib-local/src/main/scala/org/apache/spark/ml/linalg/Matrices.scala --- @@ -154,11 +172,12 @@ sealed trait Matrix extends Serializable {

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Cleanup the usel...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213739995 **[Test build #56798 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56798/consoleFull)** for PR 12638 at commit [`c79cba9`](https://g

  1   2   3   4   >