[GitHub] spark pull request: [SPARK-12616] [SQL] Adding a New Logical Opera...

2016-01-05 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48820132 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -595,6 +598,22 @@ object BooleanSimplification

[GitHub] spark pull request: [SPARK-7675][ML][PYSpark] sparkml params type ...

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

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

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

[GitHub] spark pull request: [SPARK-12570] [ML] [Doc] DecisionTreeRegressor...

2016-01-05 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/10594 [SPARK-12570] [ML] [Doc] DecisionTreeRegressor: provide variance of prediction: user guide update You can merge this pull request into a Git repository by running: $ git pull

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

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

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

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48821710 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala --- @@ -240,6 +241,23 @@ private[hive] class HiveMetastoreCatalog(val

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10595#issuecomment-168933998 Can you combine your pull requests into a single one? --- 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-12539][SQL] support writing bucketed ta...

2016-01-05 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48821401 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala --- @@ -240,6 +241,23 @@ private[hive] class

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread jaceklaskowski
GitHub user jaceklaskowski opened a pull request: https://github.com/apache/spark/pull/10595 [STREAMING][MINOR] More contextual information in logs + minor code i… …mprovements Please review and merge at your convenience. Thanks! You can merge this pull request into a

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

2016-01-05 Thread saucam
Github user saucam commented on the pull request: https://github.com/apache/spark/pull/9858#issuecomment-168934153 Fixed --- 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-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-05 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10583#issuecomment-168929936 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-7675][ML][PYSpark] sparkml params type ...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9581#issuecomment-168930028 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-7675][ML][PYSpark] sparkml params type ...

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

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

2016-01-05 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48820170 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -189,13 +220,43 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [SPARK-11944][PYSPARK][MLLIB] python mllib.clu...

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

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

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

[GitHub] spark pull request: [SPARK-11944][PYSPARK][MLLIB] python mllib.clu...

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

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

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

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

2016-01-05 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10523#issuecomment-168932900 @liancheng @yhuai Could you review this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-11944][PYSPARK][MLLIB] python mllib.clu...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10150#issuecomment-168931943 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-12640][SQL] Add simple benchmarking uti...

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

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

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48820931 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -189,13 +220,43 @@ final class DataFrameWriter private[sql](df: DataFrame)

[GitHub] spark pull request: [SPARK-12616] [SQL] Adding a New Logical Opera...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48820999 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -595,6 +598,22 @@ object BooleanSimplification

[GitHub] spark pull request: [SPARK-12393] [SparkR] Add read.text and write...

2016-01-05 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/10348#issuecomment-168933554 ping @shivaram @sun-rui @felixcheung --- 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-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48820313 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -189,13 +220,43 @@ final class DataFrameWriter private[sql](df: DataFrame)

[GitHub] spark pull request: [SPARK-12616] [SQL] Adding a New Logical Opera...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48820341 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -595,6 +598,22 @@ object BooleanSimplification extends

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

2016-01-05 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48820679 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -189,13 +220,43 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [SPARK-12480][follow-up] use a single column v...

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

[GitHub] spark pull request: [SPARK-12480][follow-up] use a single column v...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10588#issuecomment-168931720 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-12480][follow-up] use a single column v...

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

[GitHub] spark pull request: [SPARK-12644][SQL] Update parquet reader to be...

2016-01-05 Thread nongli
GitHub user nongli opened a pull request: https://github.com/apache/spark/pull/10593 [SPARK-12644][SQL] Update parquet reader to be vectorized. This inlines a few of the Parquet decoders and adds vectorized APIs to support decoding in batch. There are a few particulars in the

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

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10589#issuecomment-168932593 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-12640][SQL] Add simple benchmarking uti...

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

[GitHub] spark pull request: [SPARK-12480][follow-up] use a single column v...

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

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10595#issuecomment-16890 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12644][SQL] Update parquet reader to be...

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

[GitHub] spark pull request: [SPARK-12644][SQL] Update parquet reader to be...

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

[GitHub] spark pull request: [SPARK-12644][SQL] Update parquet reader to be...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10593#issuecomment-168934499 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-12641] Remove unused code related to Ha...

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

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

2016-01-05 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10498#discussion_r48822540 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -189,13 +220,43 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [SPARK-12570] [ML] [Doc] DecisionTreeRegressor...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10594#issuecomment-168935497 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-12401][SQL] Add integration tests for p...

2016-01-05 Thread maropu
GitHub user maropu opened a pull request: https://github.com/apache/spark/pull/10596 [SPARK-12401][SQL] Add integration tests for postgres enum types We can handle posgresql-specific enum types as strings in jdbc. So, we should just add tests and close the corresponding JIRA

[GitHub] spark pull request: [SPARK-12570] [ML] [Doc] DecisionTreeRegressor...

2016-01-05 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/10594#discussion_r48824613 --- Diff: docs/ml-classification-regression.md --- @@ -535,7 +535,9 @@ The main differences between this API and the [original MLlib Decision Tree API]

[GitHub] spark pull request: [SPARK-12625][SPARKR][SQL] replace R usage of ...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Adding a New Logical Opera...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10577#issuecomment-168943487 Todo: - Will add the new `Dataframe` and `Dataset` APIs for `unionAll`, if my understanding is correct. - Will add another rule for pushing `Filter` and

[GitHub] spark pull request: [SPARK-11579] [ML] avoid creating new optimize...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9614#issuecomment-168946249 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-11579] [ML] avoid creating new optimize...

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

[GitHub] spark pull request: [SPARK-12480][follow-up] use a single column v...

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

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9858#issuecomment-168952176 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-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10583#issuecomment-168955204 **[Test build #2322 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2322/consoleFull)** for PR 10583 at commit

[GitHub] spark pull request: [STREAMING][MINOR] Scaladoc fixes...mostly

2016-01-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10592#issuecomment-168954471 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-11624][SPARK-11972][SQL]fix commands th...

2016-01-05 Thread adrian-wang
Github user adrian-wang commented on a diff in the pull request: https://github.com/apache/spark/pull/9589#discussion_r48829056 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala --- @@ -151,29 +152,34 @@ private[hive] class ClientWrapper(

[GitHub] spark pull request: [SPARK-12453][Streaming] Remove explicit depen...

2016-01-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10492#issuecomment-168958835 Given the discussion here, I'm pretty confident in this change and would like to go ahead and merge it. It will also unblock further fixes in 12269. --- If your

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10527#issuecomment-168963321 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-12470] [SQL] Fix size reduction calcula...

2016-01-05 Thread robbinspg
Github user robbinspg commented on the pull request: https://github.com/apache/spark/pull/10421#issuecomment-168963270 I have a fix for the test failure. Should I create a new Jira and PR? --- 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-9372] [SQL] Filter nulls in Inner joins...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9451#issuecomment-168963367 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-9372] [SQL] Filter nulls in Inner joins...

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

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

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

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread jaceklaskowski
Github user jaceklaskowski commented on a diff in the pull request: https://github.com/apache/spark/pull/10595#discussion_r48830059 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala --- @@ -59,17 +59,15 @@ case class JobSet( // Time

[GitHub] spark pull request: [SPARK-12401][SQL] Add integration tests for p...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10596#issuecomment-168967487 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-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10562#issuecomment-168967549 @QiangCai I think the test failures are unrelated. However before we can retest you'll have to rebase as there is a merge conflict now. --- If your project is set up

[GitHub] spark pull request: [SPARK-12401][SQL] Add integration tests for p...

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

[GitHub] spark pull request: [SPARK-12570] [ML] [Doc] DecisionTreeRegressor...

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

[GitHub] spark pull request: [SPARK-12641] Remove unused code related to Ha...

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

[GitHub] spark pull request: [SPARK-12570] [ML] [Doc] DecisionTreeRegressor...

2016-01-05 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/10594#discussion_r48822844 --- Diff: docs/ml-classification-regression.md --- @@ -535,7 +535,9 @@ The main differences between this API and the [original MLlib Decision Tree API]

[GitHub] spark pull request: [SPARK-10658][SPARK-11421][PYSPARK][CORE] Prov...

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

[GitHub] spark pull request: [SPARK-10658][SPARK-11421][PYSPARK][CORE] Prov...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9313#issuecomment-168936339 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-10906][MLlib] More efficient SparseMatr...

2016-01-05 Thread rahulpalamuttam
Github user rahulpalamuttam commented on the pull request: https://github.com/apache/spark/pull/8960#issuecomment-168942081 @jkbradley @mengxr I have opened the new PR in Breeze here : https://github.com/scalanlp/breeze/pull/480 --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-12645] [SparkR] SparkR support hash fun...

2016-01-05 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/10597 [SPARK-12645] [SparkR] SparkR support hash function Add ```hash``` function for SparkR ```DataFrame```. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9858#issuecomment-168952173 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-12480][follow-up] use a single column v...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10588#issuecomment-168952844 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: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10595#discussion_r48828208 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/dstream/DStream.scala --- @@ -286,7 +286,7 @@ abstract class DStream[T: ClassTag] (

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10595#discussion_r48828188 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala --- @@ -59,17 +59,15 @@ case class JobSet( // Time taken to

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10595#discussion_r48828135 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala --- @@ -59,17 +59,15 @@ case class JobSet( // Time taken to

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

2016-01-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10587#issuecomment-168955492 I like this, but how about adding similar docs to other similar methods like treeAggregate, fold, etc? the semantics of fold were brought up just last week, for

[GitHub] spark pull request: [CORE][MINOR] scaladoc fixes

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

[GitHub] spark pull request: [SPARK-9372] [SQL] Filter nulls in Inner joins...

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

[GitHub] spark pull request: [STREAMING][MINOR] More contextual information...

2016-01-05 Thread jaceklaskowski
Github user jaceklaskowski commented on a diff in the pull request: https://github.com/apache/spark/pull/10595#discussion_r48830663 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala --- @@ -59,17 +59,15 @@ case class JobSet( // Time

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

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10583#issuecomment-168965051 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-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10583#issuecomment-168965050 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-12641] Remove unused code related to Ha...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10590#issuecomment-168935012 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-11579] [ML] avoid creating new optimize...

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

[GitHub] spark pull request: [SPARK-12641] Remove unused code related to Ha...

2016-01-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10590#issuecomment-168935005 I've merged this. 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

[GitHub] spark pull request: [SPARK-12641] Remove unused code related to Ha...

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

[GitHub] spark pull request: [SPARK-11579] [ML] avoid creating new optimize...

2016-01-05 Thread hhbyyh
Github user hhbyyh commented on the pull request: https://github.com/apache/spark/pull/9614#issuecomment-168935901 @avulanov Thanks for review. The only possible concern I got is LBFGSOptimizer/SGDOptimizer appears to be getter, yet they are setters. I sent an update

[GitHub] spark pull request: [SPARK-9372] [SQL] Filter nulls in Inner joins...

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

[GitHub] spark pull request: [SPARK-12393] [SparkR] Add read.text and write...

2016-01-05 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/10348#issuecomment-168938461 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-12616] [SQL] Adding a New Logical Opera...

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

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

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

[GitHub] spark pull request: [SPARK-12480][follow-up] use a single column v...

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

[GitHub] spark pull request: [STREAMING][MINOR] Scaladoc fixes...mostly

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

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

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

[GitHub] spark pull request: [SPARK-12645] [SparkR] SparkR support hash fun...

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

[GitHub] spark pull request: [SPARK-12645] [SparkR] SparkR support hash fun...

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

[GitHub] spark pull request: [SPARK-12645] [SparkR] SparkR support hash fun...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10597#issuecomment-168963977 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][MINOR] More contextual information...

2016-01-05 Thread jaceklaskowski
Github user jaceklaskowski commented on a diff in the pull request: https://github.com/apache/spark/pull/10595#discussion_r48830750 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/dstream/DStream.scala --- @@ -286,7 +286,7 @@ abstract class DStream[T: ClassTag] (

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

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

[GitHub] spark pull request: [SPARK-12401][SQL] Add integration tests for p...

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

[GitHub] spark pull request: [SPARK-12641] Remove unused code related to Ha...

2016-01-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10590 --- 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-12644][SQL] Update parquet reader to be...

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

  1   2   3   4   5   6   7   8   9   >