[GitHub] spark pull request: [SPARK-13715] [MLLIB] Remove last usages of jb...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11560#issuecomment-193297689 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13715] [MLLIB] Remove last usages of jb...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11560#issuecomment-193297683 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

[GitHub] spark pull request: [SPARK-13715] [MLLIB] Remove last usages of jb...

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

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55216537 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/ExistingRDD.scala --- @@ -226,16 +226,17 @@ private[sql] object PhysicalRDD {

[GitHub] spark pull request: [SPARK-7425] [ML] spark.ml Predictor should su...

2016-03-07 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/10355#issuecomment-193294590 @mengxr @jkbradley is there any interest in this pr or should i close it? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13640][SQL] Synchronize ScalaReflection...

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

[GitHub] spark pull request: [SPARK-11316] coalesce doesn't handle UnionRDD...

2016-03-07 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11327#issuecomment-193293645 ping @davies was there any other concern or does this look good? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-13640][SQL] Synchronize ScalaReflection...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11487#issuecomment-193293506 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-13640][SQL] Synchronize ScalaReflection...

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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

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

[GitHub] spark pull request: [SPARK-9314] [EC2] add root EBS config options...

2016-03-07 Thread nchammas
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/7647#issuecomment-193291146 We should close this PR since spark-ec2 has been moved out of this repo and to a [dedicated one](https://github.com/amplab/spark-ec2). Feel free to reopen this

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11555#discussion_r55215536 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -204,11 +207,70 @@ class SQLBuilder(logicalPlan: LogicalPlan,

[GitHub] spark pull request: [SPARK-13117] [Web UI] WebUI should use the lo...

2016-03-07 Thread devaraj-kavali
Github user devaraj-kavali commented on the pull request: https://github.com/apache/spark/pull/11490#issuecomment-193282292 Sounds fine @srowen, I will update with the change. --- 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-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55213849 --- Diff: mllib/src/test/scala/org/apache/spark/ml/source/libsvm/LibSVMRelationSuite.scala --- @@ -88,7 +88,8 @@ class LibSVMRelationSuite extends

[GitHub] spark pull request: improve the doc for "spark.memory.offHeap.size...

2016-03-07 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/11561#issuecomment-193283040 @srowen mind reviewing it? --- 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

[GitHub] spark pull request: improve the doc for "spark.memory.offHeap.size...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11561#issuecomment-193282199 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: improve the doc for "spark.memory.offHeap.size...

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

[GitHub] spark pull request: improve the doc for "spark.memory.offHeap.size...

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

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

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

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55212994 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/sources/SimpleTextRelation.scala --- @@ -1,265 +0,0 @@ -/* - * Licensed to the Apache

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

2016-03-07 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/11559#issuecomment-193279301 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 this

[GitHub] spark pull request: improve the doc for "spark.memory.offHeap.size...

2016-03-07 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/11561#issuecomment-193275908 I didn't create a JIRA as I think it is not necessaryif the reviewer still prefers with a JIRA, I can add one --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193277373 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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193277378 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

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

[GitHub] spark pull request: improve the doc for "spark.memory.offHeap.size...

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

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55211124 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala --- @@ -147,6 +147,13 @@ case class CreateMetastoreDataSource(

[GitHub] spark pull request: improve the doc for "spark.memory.offHeap.size...

2016-03-07 Thread CodingCat
GitHub user CodingCat opened a pull request: https://github.com/apache/spark/pull/11561 improve the doc for "spark.memory.offHeap.size" ## What changes were proposed in this pull request? The description of "spark.memory.offHeap.size" in the current document does not

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55210119 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala --- @@ -465,214 +379,168 @@ abstract class OutputWriter { }

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11443#issuecomment-193271862 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11443#issuecomment-193271858 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

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

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

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11559#issuecomment-193270999 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11559#issuecomment-193270998 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

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

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

[GitHub] spark pull request: [SPARK-13438][STREAMING] Remove by default das...

2016-03-07 Thread peterableda
Github user peterableda closed the pull request at: https://github.com/apache/spark/pull/11312 --- 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

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

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

[GitHub] spark pull request: [SPARK-13692][CORE][SQL] Fix trivial Coverity/...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11530#issuecomment-193262909 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13692][CORE][SQL] Fix trivial Coverity/...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11530#issuecomment-193262901 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

[GitHub] spark pull request: [SPARK-13692][CORE][SQL] Fix trivial Coverity/...

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

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

2016-03-07 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/11443#issuecomment-193259539 Finished all steps listed in the PR descriptions, trying to get all tests pass. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55206478 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -366,13 +366,6 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193257580 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193257578 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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/11555#discussion_r55205891 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -204,11 +207,70 @@ class SQLBuilder(logicalPlan: LogicalPlan,

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55205874 --- Diff: mllib/src/test/scala/org/apache/spark/ml/source/libsvm/LibSVMRelationSuite.scala --- @@ -88,7 +88,8 @@ class LibSVMRelationSuite extends

[GitHub] spark pull request: [SPARK-13665][SQL] Separate the concerns of Ha...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11509#discussion_r55204271 --- Diff: mllib/src/main/scala/org/apache/spark/ml/source/libsvm/LibSVMRelation.scala --- @@ -167,22 +117,63 @@ class DefaultSource extends

[GitHub] spark pull request: [SPARK-13599] [BUILD] remove transitive groovy...

2016-03-07 Thread steveloughran
Github user steveloughran closed the pull request at: https://github.com/apache/spark/pull/11473 --- 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 pull request: [SPARK-13640][SQL] Synchronize ScalaReflection...

2016-03-07 Thread ueshin
Github user ueshin commented on the pull request: https://github.com/apache/spark/pull/11487#issuecomment-193249061 @srowen > My question was more like, does the extra locking cause trouble in 2.11? I know you said 2.11 doesn't need this change per se. I think the extra

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

2016-03-07 Thread nchammas
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/11522#discussion_r55202635 --- Diff: dev/make-distribution.sh --- @@ -29,7 +29,7 @@ set -e set -x # Figure out where the Spark framework is installed

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

2016-03-07 Thread nchammas
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/11522#discussion_r55202112 --- Diff: dev/make-distribution.sh --- @@ -29,7 +29,7 @@ set -e set -x # Figure out where the Spark framework is installed

[GitHub] spark pull request: [SPARK-13640][SQL] Synchronize ScalaReflection...

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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193243578 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193243571 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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

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

[GitHub] spark pull request: [SPARK-13715] [MLLIB] Remove last usages of jb...

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

[GitHub] spark pull request: [SPARK-13715] [MLLIB] Remove last usages of jb...

2016-03-07 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/11560 [SPARK-13715] [MLLIB] Remove last usages of jblas in tests ## What changes were proposed in this pull request? Remove last usage of jblas, in tests ## How was this patch tested?

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

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

[GitHub] spark pull request: [SPARK-13599] [BUILD] remove transitive groovy...

2016-03-07 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11473#issuecomment-193241340 OK it's merged. You can close this PR manually now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-13117] [Web UI] WebUI should use the lo...

2016-03-07 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11490#issuecomment-193236703 Hm, maybe we should log both pieces of information. It currently logs _some_ public host name that the UI has bound to, which is good. but maybe the message can include

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11522#issuecomment-193236611 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-13596] [BUILD] Move misc top-level buil...

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

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11555#discussion_r55198266 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -204,11 +208,55 @@ class SQLBuilder(logicalPlan: LogicalPlan,

[GitHub] spark pull request: [SPARK-13117] [Web UI] WebUI should use the lo...

2016-03-07 Thread devaraj-kavali
Github user devaraj-kavali commented on a diff in the pull request: https://github.com/apache/spark/pull/11490#discussion_r5519 --- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala --- @@ -134,7 +134,8 @@ private[spark] abstract class WebUI( def bind() {

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

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

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11443#issuecomment-193233029 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11443#issuecomment-193233026 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

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

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

[GitHub] spark pull request: [SPARK-13244][SQL][WIP] Prototyping: migrates ...

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

[GitHub] spark pull request: [SPARK-13599] [BUILD] remove transitive groovy...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11473#issuecomment-193230737 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-13599] [BUILD] remove transitive groovy...

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

[GitHub] spark pull request: [SPARK-13599] [BUILD] remove transitive groovy...

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

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55195953 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ng/AstBuilder.scala --- @@ -0,0 +1,1128 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55195847 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala --- @@ -331,8 +331,8 @@ class JoinSuite extends QueryTest with SharedSQLContext {

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55195499 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala --- @@ -1953,7 +1954,6 @@ object functions extends LegacyFunctions { *

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55195410 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-13674][SQL] Add wholestage codegen supp...

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

[GitHub] spark pull request: [SPARK-13674][SQL] Add wholestage codegen supp...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11517#issuecomment-193217449 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-13674][SQL] Add wholestage codegen supp...

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

[GitHub] spark pull request: [SPARK-12718][SQL] SQL generation support for ...

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

[GitHub] spark pull request: [SPARK-13651] Generator outputs are not resolv...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11497#discussion_r55192250 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -512,6 +512,9 @@ class Analyzer( // A

[GitHub] spark pull request: [SPARK-13117] [Web UI] WebUI should use the lo...

2016-03-07 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11490#discussion_r55189207 --- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala --- @@ -134,7 +134,8 @@ private[spark] abstract class WebUI( def bind() {

[GitHub] spark pull request: Fixing the type of the sentiment happiness val...

2016-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11540 --- 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: Fixing the type of the sentiment happiness val...

2016-03-07 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11540#issuecomment-193203187 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-13232][YARN] Fix executor node label

2016-03-07 Thread steveloughran
Github user steveloughran commented on the pull request: https://github.com/apache/spark/pull/11129#issuecomment-193200945 Yarn and labels, joy. # Currently, a node can have exactly one label. That may change at a time in the future, a time called, provisionally "the patch

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55186195 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ng/ParseDriver.scala --- @@ -0,0 +1,205 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55185915 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ng/AstBuilder.scala --- @@ -0,0 +1,1128 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r55185301 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ng/AstBuilder.scala --- @@ -0,0 +1,1128 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

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

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

2016-03-07 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11522#issuecomment-193196378 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-13596] [BUILD] Move misc top-level buil...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11522#issuecomment-193195212 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

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

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

[GitHub] spark pull request: [SPARK-13596] [BUILD] Move misc top-level buil...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11522#issuecomment-193195216 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Fixing the type of the sentiment happiness val...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11540#issuecomment-193194526 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: Fixing the type of the sentiment happiness val...

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

[GitHub] spark pull request: Fixing the type of the sentiment happiness val...

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

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

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

[GitHub] spark pull request: [SPARK-13717][Core] Let RandomSampler can samp...

2016-03-07 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/11559#issuecomment-193191745 cc @mengxr @rxin @srowen --- 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

<    2   3   4   5   6   7   8   >