[GitHub] spark pull request: [SPARK-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6981#issuecomment-122198738 [Test build #37595 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37595/consoleFull) for PR 6981 at commit [`e223bc0`](https://gith

[GitHub] spark pull request: [SPARK-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/6981#discussion_r34866036 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala --- @@ -378,4 +395,183 @@ object DateTimeUtils { c.set(seg

[GitHub] spark pull request: [SPARK-4598][WebUI]Task table pagination for t...

2015-07-16 Thread carsonwang
Github user carsonwang commented on a diff in the pull request: https://github.com/apache/spark/pull/7399#discussion_r34866034 --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala --- @@ -672,164 +658,605 @@ private[ui] class StagePage(parent: StagesTab) extends W

[GitHub] spark pull request: [SPARK-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6981#issuecomment-122198594 Merged build started. --- 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-4598][WebUI]Task table pagination for t...

2015-07-16 Thread carsonwang
Github user carsonwang commented on a diff in the pull request: https://github.com/apache/spark/pull/7399#discussion_r34865990 --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala --- @@ -672,164 +658,605 @@ private[ui] class StagePage(parent: StagesTab) extends W

[GitHub] spark pull request: [SPARK-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6981#issuecomment-122198587 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-16 Thread tarekauel
Github user tarekauel commented on a diff in the pull request: https://github.com/apache/spark/pull/6981#discussion_r34865980 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala --- @@ -378,4 +395,183 @@ object DateTimeUtils { c.se

[GitHub] spark pull request: [SPARK-4598][WebUI]Task table pagination for t...

2015-07-16 Thread carsonwang
Github user carsonwang commented on a diff in the pull request: https://github.com/apache/spark/pull/7399#discussion_r34865987 --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala --- @@ -672,164 +658,605 @@ private[ui] class StagePage(parent: StagesTab) extends W

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread tarekauel
Github user tarekauel commented on a diff in the pull request: https://github.com/apache/spark/pull/7208#discussion_r34865919 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala --- @@ -593,6 +593,33 @@ case class Levenshtein(left

[GitHub] spark pull request: [SPARK-4598][WebUI]Task table pagination for t...

2015-07-16 Thread carsonwang
Github user carsonwang commented on a diff in the pull request: https://github.com/apache/spark/pull/7399#discussion_r34865792 --- Diff: core/src/main/scala/org/apache/spark/ui/PagedTable.scala --- @@ -0,0 +1,221 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122197492 [Test build #37594 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37594/consoleFull) for PR 7437 at commit [`dbf538e`](https://gith

[GitHub] spark pull request: [SPARK-8280][SPARK-8281][SQL]Handle NaN, null ...

2015-07-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7451#issuecomment-122197144 Jenkins, ok to test. --- 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 fea

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-122197190 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 project

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-122197129 [Test build #37581 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37581/console) for PR 7115 at commit [`a2d6313`](https://github.

[GitHub] spark pull request: [SPARK-9058][SQL] Split projectionCode if it i...

2015-07-16 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/7418#issuecomment-122197117 @chenghao-intel no, if there are less 50 columns, the generated java codes are as the same as the before. --- If your project is set up for it, you can reply to this ema

[GitHub] spark pull request: [SPARK-8443][SQL] Split GenerateMutableProject...

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7076#discussion_r34865652 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala --- @@ -45,7 +45,32 @@ object

[GitHub] spark pull request: [SPARK-4366] [SQL] [WIP] Aggregation Improveme...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7458#discussion_r34865653 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -537,6 +541,7 @@ class Analyzer( def containsAggrega

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122196934 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8280][SPARK-8281][SQL]Handle NaN, null ...

2015-07-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7451#issuecomment-122196900 Jenkins, add to whitelist. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have th

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122196956 Merged build started. --- 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-8443][SQL] Split GenerateMutableProject...

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7076#discussion_r34865586 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala --- @@ -45,7 +45,32 @@ object

[GitHub] spark pull request: [SPARK-9023] [SQL] [WIP] Efficiency improvemen...

2015-07-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7456#issuecomment-122195984 The current code looks pretty good to me. --- 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 d

[GitHub] spark pull request: [SPARK-8443][SQL] Split GenerateMutableProject...

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7076#discussion_r34865456 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CodeGenerationSuite.scala --- @@ -42,4 +42,8 @@ class CodeGenerationSu

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7437#discussion_r34865356 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateUnsafeProjection.scala --- @@ -0,0 +1,110 @@ +/* + * Lic

[GitHub] spark pull request: [SPARK-9023] [SQL] [WIP] Efficiency improvemen...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7456#discussion_r34865261 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala --- @@ -112,109 +114,71 @@ case class Exchange(newPartitioning: Partitioning, ch

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122191875 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 project

[GitHub] spark pull request: [SPARK-6284][MESOS] Add mesos role, principal ...

2015-07-16 Thread adam-mesos
Github user adam-mesos commented on the pull request: https://github.com/apache/spark/pull/4960#issuecomment-122191832 Thanks! :tada: :cake: :v: --- 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 ha

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122191825 [Test build #37591 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37591/console) for PR 7437 at commit [`dc737b2`](https://github.

[GitHub] spark pull request: [SPARK-6284][MESOS] Add mesos role, principal ...

2015-07-16 Thread reachbach
Github user reachbach commented on the pull request: https://github.com/apache/spark/pull/4960#issuecomment-122191175 @tnachen @andrewor14 thanks for getting this done! Much needed for mesos deployments. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-4352][YARN][WIP] Incorporate locality p...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6394#issuecomment-122191181 [Test build #37593 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37593/consoleFull) for PR 6394 at commit [`42c0f44`](https://gith

[GitHub] spark pull request: [SPARK-9058][SQL] Split projectionCode if it i...

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/7418#issuecomment-122191060 @viirya, I mean you at least have a single group (less than 50 fields), it probably cause performance issues for case like: `select a+b from src`, as the overh

[GitHub] spark pull request: [SPARK-4352][YARN][WIP] Incorporate locality p...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6394#issuecomment-122190188 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-4352][YARN][WIP] Incorporate locality p...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6394#issuecomment-122190229 Merged build started. --- 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-9058][SQL] Split projectionCode if it i...

2015-07-16 Thread viirya
Github user viirya closed the pull request at: https://github.com/apache/spark/pull/7418 --- 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 enab

[GitHub] spark pull request: [SPARK-9058][SQL] Split projectionCode if it i...

2015-07-16 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/7418#issuecomment-122189888 ok. no problem. --- 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

[GitHub] spark pull request: [SPARK-9023] [SQL] [WIP] Efficiency improvemen...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7456#discussion_r34864886 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/UnsafeRowSerializer.scala --- @@ -0,0 +1,130 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-9023] [SQL] [WIP] Efficiency improvemen...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7456#discussion_r34864869 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/UnsafeRowSerializer.scala --- @@ -0,0 +1,130 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7437#discussion_r34864786 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateUnsafeProjection.scala --- @@ -0,0 +1,113 @@ +/* + * Lic

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122189247 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 project

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122189138 [Test build #37573 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37573/console) for PR 7457 at commit [`739f5bd`](https://github.

[GitHub] spark pull request: [SPARK-8682][SQL][WIP] Range Join

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/7379#issuecomment-122188991 Sorry, I shouldn't use the word `SMJ`. I mean if we are planning to improve the performance of RangeJoin, probably we can think of it in a more general way, no

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122188919 [Test build #37592 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37592/consoleFull) for PR 7208 at commit [`05221a9`](https://gith

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7437#discussion_r34864633 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateUnsafeProjection.scala --- @@ -0,0 +1,113 @@ +/* + * L

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122188665 [Test build #37591 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37591/consoleFull) for PR 7437 at commit [`dc737b2`](https://gith

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122188112 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122188126 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122188201 Merged build started. --- 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-9022] [SQL] Generated projections for U...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7437#issuecomment-122188198 Merged build started. --- 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-9082][SQL] Filter using non-determinist...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7446#discussion_r34864521 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -513,18 +513,28 @@ object SimplifyFilters extends Rule[Logi

[GitHub] spark pull request: [SPARK-9116] [SQL] [PYSPARK] support Python UD...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7453#issuecomment-122187161 [Test build #37588 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37588/console) for PR 7453 at commit [`83d65ac`](https://github.

[GitHub] spark pull request: [SPARK-9116] [SQL] [PYSPARK] support Python UD...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7453#issuecomment-122187170 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 project

[GitHub] spark pull request: [SPARK-9022] [SQL] Generated projections for U...

2015-07-16 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7437#discussion_r34864461 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/UnsafeRowConverter.scala --- @@ -261,6 +265,13 @@ private class StringUnsafeCo

[GitHub] spark pull request: [SPARK-9058][SQL] Split projectionCode if it i...

2015-07-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7418#issuecomment-122186634 Thanks for understanding. Do you mind closing this one first? If we need to merge this one, let's reopen it. --- If your project is set up for it, you can reply to this e

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7457 --- 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 enab

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122186121 Merged into master. Thanks for reviewing! --- 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-8269][SQL]string function: initcap

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122185720 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 project

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122185717 [Test build #37590 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37590/console) for PR 7208 at commit [`fcb4f90`](https://github.

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122185196 [Test build #37590 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37590/consoleFull) for PR 7208 at commit [`fcb4f90`](https://gith

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122184902 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 project

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122184739 [Test build #37585 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37585/console) for PR 7457 at commit [`4b40faa`](https://github.

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122184731 Merged build started. --- 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-9117][SQL] fix BooleanSimplification in...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7452#issuecomment-122184778 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 project

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-122184681 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-9117][SQL] fix BooleanSimplification in...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7452#issuecomment-122184575 [Test build #37578 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37578/console) for PR 7452 at commit [`2a6e692`](https://github.

[GitHub] spark pull request: [SPARK-9116] [SQL] [PYSPARK] support Python UD...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7453#issuecomment-122184101 [Test build #37588 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37588/consoleFull) for PR 7453 at commit [`83d65ac`](https://gith

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122183530 [Test build #37589 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37589/consoleFull) for PR 6756 at commit [`a14939b`](https://gith

[GitHub] spark pull request: [SPARK-9058][SQL] Split projectionCode if it i...

2015-07-16 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/7418#issuecomment-122183508 @rxin sure, no problem. --- 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-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122183394 Merged build started. --- 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-9116] [SQL] [PYSPARK] support Python UD...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7453#issuecomment-122183375 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122183383 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-9116] [SQL] [PYSPARK] support Python UD...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7453#issuecomment-122183395 Merged build started. --- 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-9052][SparkR] Fix comments after curly ...

2015-07-16 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/7440#discussion_r34863617 --- Diff: R/pkg/R/schema.R --- @@ -69,10 +69,11 @@ structType.structField <- function(x, ...) { #' @param ... further arguments passed to or from other me

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122182463 [Test build #37587 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37587/consoleFull) for PR 6756 at commit [`4c61693`](https://gith

[GitHub] spark pull request: [SPARK-8682][SQL][WIP] Range Join

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7379#discussion_r34863539 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/BroadcastRangeJoin.scala --- @@ -0,0 +1,411 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122182225 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-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122182129 Merged build started. --- 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-9052][SparkR] Fix comments after curly ...

2015-07-16 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/7440#discussion_r34863449 --- Diff: R/pkg/R/schema.R --- @@ -69,10 +69,11 @@ structType.structField <- function(x, ...) { #' @param ... further arguments passed to or from other me

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122182121 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-9113][SQL] enable analysis check code f...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7449#issuecomment-122181790 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 project

[GitHub] spark pull request: [SPARK-9113][SQL] enable analysis check code f...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7449#issuecomment-122181763 [Test build #37575 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37575/console) for PR 7449 at commit [`07385df`](https://github.

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-16 Thread HuJiayin
Github user HuJiayin commented on a diff in the pull request: https://github.com/apache/spark/pull/7208#discussion_r34863266 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala --- @@ -552,6 +552,34 @@ case class Substring(str: Ex

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122180218 LGTM but maybe wait for Patrick to be sure --- 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 proj

[GitHub] spark pull request: [SPARK-8996] [MLlib] [PySpark] Python API for ...

2015-07-16 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/7430#discussion_r34863180 --- Diff: python/pyspark/mllib/stat/_statistics.py --- @@ -238,6 +242,54 @@ def chiSqTest(observed, expected=None): jmodel = callMLlibFunc("chi

[GitHub] spark pull request: [SPARK-8996] [MLlib] [PySpark] Python API for ...

2015-07-16 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7430#issuecomment-122179544 @josepablocam @MechCoder It should be sufficient to support only the `distName` + `params` API in this PR. --- If your project is set up for it, you can reply to this em

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread yu-iskw
Github user yu-iskw commented on a diff in the pull request: https://github.com/apache/spark/pull/6756#discussion_r34863112 --- Diff: python/docs/pyspark.ml.rst --- @@ -33,6 +33,14 @@ pyspark.ml.classification module :undoc-members: :inherited-members: +p

[GitHub] spark pull request: [SPARK-7131] [ml] Copy Decision Tree, Random F...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7294 --- 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 enab

[GitHub] spark pull request: [SPARK-7131] [ml] Copy Decision Tree, Random F...

2015-07-16 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7294#issuecomment-122178963 Merged into master. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-9113][SQL] enable analysis check code f...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7449#issuecomment-122178815 [Test build #37586 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37586/consoleFull) for PR 7449 at commit [`2bb6637`](https://gith

[GitHub] spark pull request: [SPARK-7131] [ml] Copy Decision Tree, Random F...

2015-07-16 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7294#issuecomment-122178873 @jkbradley Just thought about how to reduce the review complexity. I think this PR would be easier to review if you copied the implementation with bins and kept the 1-1 m

[GitHub] spark pull request: [SPARK-4352][YARN][WIP] Incorporate locality p...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6394#issuecomment-122178682 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 project

[GitHub] spark pull request: [SPARK-9052][SparkR] Fix comments after curly ...

2015-07-16 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/7440#discussion_r34862903 --- Diff: R/pkg/R/utils.R --- @@ -440,19 +446,23 @@ processClosure <- function(node, oldEnv, defVars, checkedFuncs, newEnv) { # Namespaces other

[GitHub] spark pull request: [SPARK-4352][YARN][WIP] Incorporate locality p...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6394#issuecomment-122178646 [Test build #31 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/31/console) for PR 6394 at commit [`f18dc18`](https://github.co

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122178572 [Test build #37585 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37585/consoleFull) for PR 7457 at commit [`4b40faa`](https://gith

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122178454 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-9113][SQL] enable analysis check code f...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7449#issuecomment-122178469 Merged build started. --- 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-9113][SQL] enable analysis check code f...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7449#issuecomment-122178457 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-9126][MLLIB] do not assert on time take...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7457#issuecomment-122178465 Merged build started. --- 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-9052][SparkR] Fix comments after curly ...

2015-07-16 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/7440#discussion_r34862787 --- Diff: R/pkg/R/schema.R --- @@ -69,10 +69,11 @@ structType.structField <- function(x, ...) { #' @param ... further arguments passed to or from other me

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-16 Thread yu-iskw
Github user yu-iskw commented on a diff in the pull request: https://github.com/apache/spark/pull/6756#discussion_r34862779 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala --- @@ -0,0 +1,204 @@ +/* + * Licensed to the Apache Software Foundation (A

[GitHub] spark pull request: [SPARK-5681][Streaming] Move 'stopReceivers' t...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6294#issuecomment-122178072 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 project

  1   2   3   4   5   6   7   8   9   10   >