[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

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

[GitHub] spark pull request: [SPARK-12581][SQL] Support case-sensitive tabl...

2016-01-06 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10523#issuecomment-169284609 @yhuai yes and it's a pg-specific thing. I agree that we add it as a data source option instead of sql conf. If the fix is acceptable, I'll fix this pr to add a data

[GitHub] spark pull request: [SPARK-12632][Python][Make Parameter Descripti...

2016-01-06 Thread vijaykiran
Github user vijaykiran commented on a diff in the pull request: https://github.com/apache/spark/pull/10602#discussion_r48945561 --- Diff: python/pyspark/mllib/fpm.py --- @@ -67,12 +67,15 @@ class FPGrowth(object): def train(cls, data, minSupport=0.3, numPartitions=-1):

[GitHub] spark pull request: [SPARK-4819] [WIP] Remove Guava's "Optional" f...

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

[GitHub] spark pull request: [SPARK-4819] [WIP] Remove Guava's "Optional" f...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10513#issuecomment-169302078 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-4819] [WIP] Remove Guava's "Optional" f...

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

[GitHub] spark pull request: [SPARK-12340][SQL]fix Int overflow in the Spar...

2016-01-06 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/10562#issuecomment-169274323 @QiangCai We have many conflicts against `branch-1.6` so I'd merge this into only `master` for now. If you want merge this into `branch-1.6` please feel free open

[GitHub] spark pull request: [SPARK-12603] [MLlib] PySpark MLlib GaussianMi...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10552#issuecomment-169273957 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-12603] [MLlib] PySpark MLlib GaussianMi...

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

[GitHub] spark pull request: [SPARK-12630][DOC] Update param descriptions

2016-01-06 Thread vijaykiran
Github user vijaykiran commented on the pull request: https://github.com/apache/spark/pull/10598#issuecomment-169275901 @BryanCutler Thanks for the review, I added a new commit - will review update other two PRs with 100 fill-column as well. --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-12673][UI] Add missing uri prepending f...

2016-01-06 Thread jerryshao
GitHub user jerryshao opened a pull request: https://github.com/apache/spark/pull/10618 [SPARK-12673][UI] Add missing uri prepending for job description Otherwise the url will be failed to proxy to the right one if in YARN mode. Here is the screenshot: ![screen shot

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48940817 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/parser/ParseUtils.java --- @@ -0,0 +1,163 @@ +/** + * Licensed to the Apache

[GitHub] spark pull request: [STREAMING][DOCS][EXAMPLES] Minor fixes

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

[GitHub] spark pull request: SPARK-12637 Print stage info of finished stage...

2016-01-06 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10585#discussion_r48936240 --- Diff: core/src/main/scala/org/apache/spark/scheduler/StageInfo.scala --- @@ -61,6 +61,21 @@ class StageInfo( "running" } }

[GitHub] spark pull request: [Streaming][UI][SPARK-12672]Use the uiRoot fun...

2016-01-06 Thread SaintBacchus
GitHub user SaintBacchus opened a pull request: https://github.com/apache/spark/pull/10617 [Streaming][UI][SPARK-12672]Use the uiRoot function instead of default root path to gain the streaming batch url. You can merge this pull request into a Git repository by running: $

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

2016-01-06 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10554#discussion_r48938992 --- Diff: core/src/main/scala/org/apache/spark/api/java/JavaPairRDD.scala --- @@ -288,17 +288,18 @@ class JavaPairRDD[K, V](val rdd: RDD[(K, V)]) *

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

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

[GitHub] spark pull request: [STREAMING][DOCS][EXAMPLES] Minor fixes

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

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

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

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10570#issuecomment-169281770 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-12420][SQL] Have a built-in CSV data so...

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

[GitHub] spark pull request: [SPARK-12673][UI] Add missing uri prepending f...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10618#issuecomment-169302612 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-12673][UI] Add missing uri prepending f...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48948491 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -1212,7 +1212,12 @@

[GitHub] spark pull request: [SPARK-12665][CORE][GRAPHX] Remove Vector, Vec...

2016-01-06 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/10613#issuecomment-169266714 Thanks for your suggestion. I've update the title and corresponding JIRA. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

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

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

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

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

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

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10615#issuecomment-169285300 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-12539][SQL] support writing bucketed ta...

2016-01-06 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48951109 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/WriterContainer.scala --- @@ -335,6 +339,117 @@ private[sql] class

[GitHub] spark pull request: [SPARK-12578][SQL] Distinct should not be sile...

2016-01-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10557 --- 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-12591][Streaming]Register OpenHashMapBa...

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

[GitHub] spark pull request: [SPARK-12591][Streaming]Register OpenHashMapBa...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10609#issuecomment-169270414 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: [Streaming][UI][SPARK-12672]Use the uiRoot fun...

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

[GitHub] spark pull request: [SPARK-12340][SQL]fix Int overflow in the Spar...

2016-01-06 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/10562#issuecomment-169272584 Merging this into `master` and `branch-1.6`. Thanks @QiangCai ! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-12665][CORE][GRAPHX] Remove Vector, Vec...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10613#issuecomment-169277161 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-12665][CORE][GRAPHX] Remove Vector, Vec...

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

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

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

[GitHub] spark pull request: [SPARK-12340][SQL]fix Int overflow in the Spar...

2016-01-06 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10562#issuecomment-169270903 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

[GitHub] spark pull request: [SPARK-5682][Core] Add encrypted shuffle in sp...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8880#issuecomment-169271043 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-12638] [API DOC] Parameter explanation ...

2016-01-06 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10587#issuecomment-169270733 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

[GitHub] spark pull request: [SPARK-5682][Core] Add encrypted shuffle in sp...

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

[GitHub] spark pull request: [SPARK-12340][SQL]fix Int overflow in the Spar...

2016-01-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10562 --- 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-12673][UI] Add missing uri prepending f...

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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

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

[GitHub] spark pull request: [SPARK-12603] [MLlib] PySpark MLlib GaussianMi...

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

[GitHub] spark pull request: [SPARK-12578][SQL] Distinct should not be sile...

2016-01-06 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10557#issuecomment-169267439 Merged. 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 this

[GitHub] spark pull request: [SPARK-5682][Core] Add encrypted shuffle in sp...

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

[GitHub] spark pull request: [SPARK-12603] [MLlib] PySpark MLlib GaussianMi...

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

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

2016-01-06 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10570#issuecomment-169277879 (Nevermind -- I backed out the `commons-codec` change for now.) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-12673][UI] Add missing uri prepending f...

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

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10615#issuecomment-169285920 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-12591][Streaming]Register OpenHashMapBa...

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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48940364 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala --- @@ -0,0 +1,969 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-12665][CORE][GRAPHX] Remove Vector, Vec...

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

[GitHub] spark pull request: [Streaming][UI][SPARK-12672]Use the uiRoot fun...

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

[GitHub] spark pull request: [Streaming][UI][SPARK-12672]Use the uiRoot fun...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10617#issuecomment-169281563 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-4819] [WIP] Remove Guava's "Optional" f...

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

[GitHub] spark pull request: [Streaming][UI][SPARK-12672]Use the uiRoot fun...

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

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10498#issuecomment-169317846 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-12539][SQL] support writing bucketed ta...

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

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

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

[GitHub] spark pull request: [SPARK-12649][SQL][WIP] support reading bucket...

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

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

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

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

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

[GitHub] spark pull request: [SPARK-12649][SQL][WIP] support reading bucket...

2016-01-06 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10604#issuecomment-169330013 you can review the last commit to ignore bucket write part. --- 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-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48951342 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -451,6 +452,19 @@ private[spark] object SQLConf { doc = "When true,

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

2016-01-06 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48951242 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala --- @@ -351,7 +365,18 @@ abstract class OutputWriterFactory extends

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48951594 --- Diff: dev/deps/spark-deps-hadoop-2.2 --- @@ -5,8 +5,7 @@ activation-1.1.jar akka-actor_2.10-2.3.11.jar akka-remote_2.10-2.3.11.jar

[GitHub] spark pull request: [SPARK-12006][ML][PYTHON] Fix GMM failure if i...

2016-01-06 Thread zero323
Github user zero323 commented on the pull request: https://github.com/apache/spark/pull/9986#issuecomment-169327083 @thunterdb Done. I am not sure if a separate test is justified. It could be better to add a second round to `test_gmm`. --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

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

[GitHub] spark pull request: [SPARK-4819] [WIP] Remove Guava's "Optional" f...

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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48951642 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala --- @@ -0,0 +1,961 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

2016-01-06 Thread steveloughran
Github user steveloughran commented on the pull request: https://github.com/apache/spark/pull/10615#issuecomment-169314710 Is this going to require the new parser JAR on the classpath everywhere, or will everything excluding CSV parsing still work without it? --- If your project is

[GitHub] spark pull request: [SPARK-12638] [API DOC] Parameter explanation ...

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

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

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

[GitHub] spark pull request: [SPARK-12673][UI] Add missing uri prepending f...

2016-01-06 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/10618#issuecomment-169323843 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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

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

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10498#issuecomment-169313888 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-12539][SQL] support writing bucketed ta...

2016-01-06 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48953481 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/WriterContainer.scala --- @@ -335,6 +339,121 @@ private[sql] class

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10583#issuecomment-169326441 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-12604] [CORE] Java count(AprroxDistinct...

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

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10554#issuecomment-169328832 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-12638] [API DOC] Parameter explanation ...

2016-01-06 Thread Wenpei
Github user Wenpei commented on the pull request: https://github.com/apache/spark/pull/10587#issuecomment-169315605 thanks,@srowen @rxin There is no Jenkins report, anything need for me? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

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

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

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

[GitHub] spark pull request: [SPARK-10149] [CORE] [WIP] Locality Level is a...

2016-01-06 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/8533#discussion_r48953313 --- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskLocation.scala --- @@ -35,16 +37,28 @@ case class ExecutorCacheTaskLocation(override val

[GitHub] spark pull request: [SPARK-12638] [API DOC] Parameter explanation ...

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

[GitHub] spark pull request: [SPARK-12673][UI] Add missing uri prepending f...

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

[GitHub] spark pull request: [SPARK-12640][SQL] Add simple benchmarking uti...

2016-01-06 Thread kiszk
Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/10589#discussion_r48960725 --- Diff: core/src/test/scala/org/apache/spark/Benchmark.scala --- @@ -0,0 +1,102 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] spark pull request: [SPARK-12539][SQL] support writing bucketed ta...

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

[GitHub] spark pull request: [SPARK-12295] [SQL] external spilling for wind...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10605#discussion_r48966227 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala --- @@ -307,27 +314,63 @@ case class Window( // Collect all

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10570#issuecomment-169362292 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-12340][SQL]fix Int overflow in the Spar...

2016-01-06 Thread QiangCai
Github user QiangCai commented on the pull request: https://github.com/apache/spark/pull/10562#issuecomment-169369194 OK. I have created another PR https://github.com/apache/spark/pull/10619. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12593][SQL] Converts resolved logical p...

2016-01-06 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-169372469 @rxin SQL generation dedicated test cases are added in `HiveQLSQLBuilderSuite`. Bugs found by round-trip tests in `HiveCompatibilitySuite` are covered there. ---

[GitHub] spark pull request: [SPARK-12649][SQL][WIP] support reading bucket...

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

[GitHub] spark pull request: [SPARK-12295] [SQL] external spilling for wind...

2016-01-06 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10605#discussion_r48967411 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala --- @@ -283,23 +287,26 @@ case class Window( val grouping =

[GitHub] spark pull request: [SPARK-12618] [CORE] [STREAMING] [SQL] Clean u...

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

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

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

  1   2   3   4   5   6   7   >