[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-122031079 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

[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-122031100 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

[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-122031320 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: [WIP][SPARK-7018][Build]: Refactor dev/run-tes...

2015-07-16 Thread brennonyork
Github user brennonyork commented on the pull request: https://github.com/apache/spark/pull/7401#issuecomment-122031700 Its worthwhile to note as well that this patch will consume and resolve [SPARK-6557](https://issues.apache.org/jira/browse/SPARK-6557) as well. --- If your project

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

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

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7342#issuecomment-122033321 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-5681][Streaming] Move 'stopReceivers' t...

2015-07-16 Thread harishreedharan
Github user harishreedharan commented on the pull request: https://github.com/apache/spark/pull/6294#issuecomment-122033912 @zsxwing Could you post a bit more detail on the offline discussion please? The last update from @tdas suggests that this is no longer required - so some

[GitHub] spark pull request: [SPARK-8464][Core][Shuffle] Consider separatin...

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

[GitHub] spark pull request: [SPARK-8464][Core][Shuffle] Consider separatin...

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

[GitHub] spark pull request: [SPARK-8297] [YARN] Scheduler backend is not n...

2015-07-16 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/7431#issuecomment-122034172 I can try it out (I'll also add some logging in my internal build to make sure the code is actually being exercised). --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-8464][Core][Shuffle] Consider separatin...

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

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

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

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

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

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

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

[GitHub] spark pull request: [SPARK-8644] Include call site in SparkExcepti...

2015-07-16 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7028#issuecomment-122035252 lgtm ! thanks for exploring all these options, I like this solution, huge improvement over the current behavior --- If your project is set up for it, you can reply to

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

2015-07-16 Thread josepablocam
Github user josepablocam commented on the pull request: https://github.com/apache/spark/pull/7430#issuecomment-122035440 @MechCoder I'm not as familiar with Python, so this might be a horrid suggestion, but one option would be to require users to pass in a named function, rather than

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7438#issuecomment-122035423 [Test build #27 has started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/27/consoleFull) for PR 7438 at commit

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

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

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

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

[GitHub] spark pull request: [SPARK-8857][SPARK-8859][Core]Add an internal ...

2015-07-16 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7448#issuecomment-122035925 What happens if my accumulator update is huge? Could this lead to dropped heartbeats? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-8857][SPARK-8859][Core]Add an internal ...

2015-07-16 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7448#issuecomment-122035997 Ah, I see now that it's only for specific accumulators and is internal-only. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

2015-07-16 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/7438#discussion_r34817673 --- Diff: python/pyspark/context.py --- @@ -152,6 +152,11 @@ def _do_init(self, master, appName, sparkHome, pyFiles, environment, batchSize,

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

2015-07-16 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/7438#issuecomment-122036240 LGTM since I can't think of a better solution. --- 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-6941][SQL] Provide a better error messa...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7342 --- 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-8899][SQL] remove duplicated equals met...

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

[GitHub] spark pull request: [SPARK-8646]PySpark does not run on YARN if ma...

2015-07-16 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/7438#discussion_r34817168 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -616,7 +616,8 @@ private[spark] class Client( val appId =

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

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

[GitHub] spark pull request: [SPARK-8899][SQL] remove duplicated equals met...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7291#issuecomment-122036367 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-8996] [MLlib] [PySpark] Python API for ...

2015-07-16 Thread MechCoder
Github user MechCoder commented on the pull request: https://github.com/apache/spark/pull/7430#issuecomment-122036351 Hmm. I did try that out, but there was some problem with python to java conversion. I'll have a deeper look. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7342#issuecomment-122036596 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-6941][SQL] Provide a better error messa...

2015-07-16 Thread yijieshen
Github user yijieshen commented on the pull request: https://github.com/apache/spark/pull/7342#issuecomment-122036783 @yhuai, Thanks for the step by step guide, really appreciate it. --- 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-9015] [BUILD] Clean project import in s...

2015-07-16 Thread jendap
Github user jendap commented on the pull request: https://github.com/apache/spark/pull/7375#issuecomment-122036687 thanks @srowen --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark pull request: [SPARK-8103][core] DAGScheduler should not sub...

2015-07-16 Thread kayousterhout
Github user kayousterhout commented on the pull request: https://github.com/apache/spark/pull/6750#issuecomment-122113535 I had one last idea for how to simplify this: https://github.com/squito/spark/pull/4, that I made sure doesn't break any tests this time! Let me know what your

[GitHub] spark pull request: [SPARK-8119] HeartbeatReceiver should replace ...

2015-07-16 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/7107#issuecomment-122115691 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-8764][ML] string indexer should take op...

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

[GitHub] spark pull request: [SPARK-8638] [SQL] Window Function Performance...

2015-07-16 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/7057#discussion_r34844383 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala --- @@ -38,443 +84,661 @@ case class Window( child: SparkPlan)

[GitHub] spark pull request: [SPARK-8638] [SQL] Window Function Performance...

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

[GitHub] spark pull request: [SPARK-8638] [SQL] Window Function Performance...

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

[GitHub] spark pull request: [SPARK-8481] [MLlib] GaussianMixtureModel pred...

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

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] Binary processing so...

2015-07-16 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/6444#discussion_r34847593 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillWriter.java --- @@ -0,0 +1,146 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-8638] [SQL] Window Function Performance...

2015-07-16 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7057#discussion_r34847515 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/WindowSuite.scala --- @@ -0,0 +1,79 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [MLlib]OnlineLDA Performance Improvements

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

[GitHub] spark pull request: [MLlib]OnlineLDA Performance Improvements

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

[GitHub] spark pull request: [SPARK-8638] [SQL] Window Function Performance...

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

[GitHub] spark pull request: [MLlib]OnlineLDA Performance Improvements

2015-07-16 Thread feynmanliang
GitHub user feynmanliang opened a pull request: https://github.com/apache/spark/pull/7454 [MLlib]OnlineLDA Performance Improvements Use range-slicing (coalesced memory access), in-place updates, and reduce number of transposes in OnlineLDA implementation. You can merge this pull

[GitHub] spark pull request: [SPARK-9100] [SQL] Adds DataFrame reader/write...

2015-07-16 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/7444#discussion_r34852485 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala --- @@ -268,6 +268,14 @@ class DataFrameReader private[sql](sqlContext:

[GitHub] spark pull request: [SPARK-9114] [SQL] [PySpark] convert returned ...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7450#issuecomment-122135446 [Test build #1093 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1093/consoleFull) for PR 7450 at commit

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

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

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

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

[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-122136644 [Test build #1094 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1094/consoleFull) for PR 7453 at commit

[GitHub] spark pull request: [SPARK-4176] [SQL] Supports decimal types with...

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

[GitHub] spark pull request: [MLlib]OnlineLDA Performance Improvements

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7454#issuecomment-122138072 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-4176] [SQL] Supports decimal types with...

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

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

[GitHub] spark pull request: [SPARK-8857][SPARK-8859][Core]Add an internal ...

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

[GitHub] spark pull request: [SPARK-8851][YARN] In Client mode, make sure t...

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

[GitHub] spark pull request: [SPARK-8851][YARN] In Client mode, make sure t...

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

[GitHub] spark pull request: [WIP][SPARK-7018][Build]: Refactor dev/run-tes...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7401#issuecomment-122055838 **[Test build #37532 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37532/console)** for PR 7401 at commit

[GitHub] spark pull request: [SPARK-8774] [ML] Add R model formula with bas...

2015-07-16 Thread ericl
Github user ericl commented on a diff in the pull request: https://github.com/apache/spark/pull/7381#discussion_r34830515 --- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/RModelFormula.scala --- @@ -0,0 +1,136 @@ +/* + * Licensed to the Apache Software Foundation

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

2015-07-16 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7452#issuecomment-122069909 cc @marmbrus @chenghao-intel --- 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-9117][SQL] fix BooleanSimplification in...

2015-07-16 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7452#discussion_r34830499 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala --- @@ -40,29 +40,11 @@ class

[GitHub] spark pull request: [SPARK-8774] [ML] Add R model formula with bas...

2015-07-16 Thread ericl
Github user ericl commented on the pull request: https://github.com/apache/spark/pull/7381#issuecomment-122069902 Sounds good, I'll look at the R integration next. --- 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-9026] Refactor SimpleFutureAction.onCom...

2015-07-16 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/7385#issuecomment-122070057 Seems unrelated random hang. Trying to run this again. --- 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-9026] Refactor SimpleFutureAction.onCom...

2015-07-16 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/7385#issuecomment-122070083 Jenkins, test 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: [SQL][minor] use lazy val instead of do the in...

2015-07-16 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7372#issuecomment-122070564 Alright agreed, should leave it to the future expression init stuff, closing. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SQL][minor] use lazy val instead of do the in...

2015-07-16 Thread cloud-fan
Github user cloud-fan closed the pull request at: https://github.com/apache/spark/pull/7372 --- 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-8464][Core][Shuffle] Consider separatin...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7129#issuecomment-122074664 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-8875] Remove BlockStoreShuffleFetcher c...

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

[GitHub] spark pull request: [SPARK-8875] Remove BlockStoreShuffleFetcher c...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7268#issuecomment-122078479 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-8064, build against Hive 1.2.1

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

[GitHub] spark pull request: SPARK-8064, build against Hive 1.2.1

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

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

2015-07-16 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/7356#issuecomment-121848726 @viirya, 1. Why pass the lower bound and upper bound in a vector instead of pass separately? They are passed separately in Scala API. and A vector can not hold

[GitHub] spark pull request: [SPARK-8995][SQL] cast date strings like '2015...

2015-07-16 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7353#discussion_r34758993 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala --- @@ -180,4 +182,202 @@ object DateTimeUtils { val

[GitHub] spark pull request: [SPARK-8995][SQL] cast date strings like '2015...

2015-07-16 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7353#discussion_r34758673 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeUtilsSuite.scala --- @@ -86,4 +88,198 @@ class DateTimeUtilsSuite extends

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

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

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

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

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

2015-07-16 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-121844421 @HuJiayin you need to rebase the code. --- 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-8807][SparkR] Add between operator in S...

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

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7356#issuecomment-121844558 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-8972][SQL]Incorrect result for rollup

2015-07-16 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/7343#issuecomment-121846486 LGTM. I am merging it to master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8995][SQL] cast date strings like '2015...

2015-07-16 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/7353#issuecomment-121846456 LGTM, just some minor comments. --- 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-8995][SQL] cast date strings like '2015...

2015-07-16 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7353#discussion_r34759205 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala --- @@ -180,4 +182,202 @@ object DateTimeUtils { val

[GitHub] spark pull request: [SPARK-9085][SQL] Remove LeafNode, UnaryNode, ...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7434#discussion_r34759850 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -277,15 +276,21 @@ abstract class LogicalPlan

[GitHub] spark pull request: [SPARK-9085][SQL] Remove LeafNode, UnaryNode, ...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7434#discussion_r34759808 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala --- @@ -96,7 +95,7 @@ case class UnresolvedFunction(name: String,

[GitHub] spark pull request: [SPARK-9085][SQL] Remove LeafNode, UnaryNode, ...

2015-07-16 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7434#discussion_r34759860 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -277,15 +276,21 @@ abstract class LogicalPlan

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

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

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

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

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

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7418#issuecomment-121843429 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-7422][MLLIB] Add argmax to Vector, Spar...

2015-07-16 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/6112#issuecomment-121849815 @GeorgeDittmar I will try to fix it and send you a PR. --- 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-5989] [MLlib] Model save/load for LDA

2015-07-16 Thread MechCoder
Github user MechCoder commented on a diff in the pull request: https://github.com/apache/spark/pull/6948#discussion_r34759686 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/LDAModel.scala --- @@ -184,6 +199,82 @@ class LocalLDAModel private[clustering] (

[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-121842954 @jkbradley I made one pass: 1. Some of my comments also apply to the old implementation. So I don't think it is necessary to address them in this PR. 2. It

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

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

[GitHub] spark pull request: [SPARK-8972][SQL]Incorrect result for rollup

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7343 --- 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-4352][YARN][WIP] Incorporate locality p...

2015-07-16 Thread jerryshao
Github user jerryshao commented on a diff in the pull request: https://github.com/apache/spark/pull/6394#discussion_r34758587 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -872,6 +872,25 @@ class DAGScheduler( // will be posted, which

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

2015-07-16 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/7356#issuecomment-121844789 Thanks @viirya -- 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

[GitHub] spark pull request: [SPARK-8995][SQL] cast date strings like '2015...

2015-07-16 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7353#discussion_r34759109 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala --- @@ -180,4 +182,202 @@ object DateTimeUtils { val

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

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

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

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

[GitHub] spark pull request: [SPARK-8807][SparkR] Add between operator in S...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7356 --- 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-9066][SQL] Improve cartesian performanc...

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

  1   2   3   4   5   6   7   8   9   10   >