[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163144631 **[Test build #47417 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47417/consoleFull)** for PR 10217 at commit [`92b1bd7`](https://g

[GitHub] spark pull request: [SPARK-12161][SQL] Ignore order of predicates ...

2015-12-08 Thread codingjaguar
Github user codingjaguar commented on the pull request: https://github.com/apache/spark/pull/10163#issuecomment-163144207 In last change we deleted `equivalentPredicate` and moved the functionality to `Expression.semanticEquals`. --- If your project is set up for it, you can reply t

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/9659#discussion_r47059656 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala --- @@ -50,27 +50,33 @@ class ParquetFilter

[GitHub] spark pull request: [SPARK-12236][SQL] JDBC filter tests all pass ...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10221#issuecomment-163143964 **[Test build #47418 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47418/consoleFull)** for PR 10221 at commit [`3294b31`](https://gi

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163143093 also could you state in the method description that subset parameter is optional? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread sun-rui
Github user sun-rui commented on a diff in the pull request: https://github.com/apache/spark/pull/10217#discussion_r47059514 --- Diff: R/pkg/R/DataFrame.R --- @@ -1206,10 +1206,15 @@ setMethod("[", signature(x = "DataFrame", i = "Column"), #' df[df$age %in% c(19, 30), 1:2]

[GitHub] spark pull request: [SPARK-12235][SPARKR] Enhance mutate() to supp...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10220#issuecomment-163142448 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 projec

[GitHub] spark pull request: [SPARK-12235][SPARKR] Enhance mutate() to supp...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10220#issuecomment-163142450 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12235][SPARKR] Enhance mutate() to supp...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10220#issuecomment-163142357 **[Test build #47414 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47414/consoleFull)** for PR 10220 at commit [`ac34da1`](https://g

[GitHub] spark pull request: [SPARK-12202][REPL] Parse Scala command line a...

2015-12-08 Thread dragos
Github user dragos closed the pull request at: https://github.com/apache/spark/pull/10199 --- 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 ena

[GitHub] spark pull request: [SPARK-10180] [SQL] JDBC datasource are not pr...

2015-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/8743#issuecomment-163142135 Just a question. Now it looks the PR for the end-to-end docker tests is merged. Do you think it needs all the tests for all the databases (namely `MySQLIntegrationS

[GitHub] spark pull request: [SPARK-11832] [Core] Process arguments in spar...

2015-12-08 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/9824#issuecomment-163141839 @jodersky, I didn't see you PR and opened #10199. I think it makes sense to push yours and close mine, but it would be good to treat errors and stop execution in that cas

[GitHub] spark pull request: [SPARK-12202][REPL] Parse Scala command line a...

2015-12-08 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/10199#issuecomment-163141579 Oh, I missed that one. Yes, it's the same. I also show errors if any of the arguments doesn't parse, and stop execution in that case, but there's no reason this can't go

[GitHub] spark pull request: [SPARK-12236][SQL] JDBC filter tests all pass ...

2015-12-08 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/10221 [SPARK-12236][SQL] JDBC filter tests all pass if filters are not really pushed down https://issues.apache.org/jira/browse/SPARK-12236 Currently JDBC filters are not tested properly. All the

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163141222 **[Test build #47417 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47417/consoleFull)** for PR 10217 at commit [`92b1bd7`](https://gi

[GitHub] spark pull request: [SPARK-12145][SQL] Command 'Set Role [ADMIN|NO...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10144#issuecomment-163141064 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 projec

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

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8880#issuecomment-163141068 **[Test build #47416 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47416/consoleFull)** for PR 8880 at commit [`eb0632d`](https://gith

[GitHub] spark pull request: [SPARK-12145][SQL] Command 'Set Role [ADMIN|NO...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10144#issuecomment-163141027 **[Test build #47410 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47410/consoleFull)** for PR 10144 at commit [`d3498f9`](https://g

[GitHub] spark pull request: [SPARK-12145][SQL] Command 'Set Role [ADMIN|NO...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10144#issuecomment-163141066 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12131][SQL] Fixes ExpressionEncoder for...

2015-12-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/10133#discussion_r47058196 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects.scala --- @@ -359,10 +359,27 @@ case class LambdaVariable(value: S

[GitHub] spark pull request: [SPARK-12235][SPARKR] Enhance mutate() to supp...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10220#issuecomment-163139798 **[Test build #47414 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47414/consoleFull)** for PR 10220 at commit [`ac34da1`](https://gi

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/10217#discussion_r47057860 --- Diff: R/pkg/R/DataFrame.R --- @@ -1209,7 +1209,11 @@ setMethod("[", signature(x = "DataFrame", i = "Column"), #' } setMethod("subset", sign

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163138323 cool, looks good --- 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-11551][DOC] Replace example code in ml-...

2015-12-08 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/10219#issuecomment-163138018 Ping @mengxr, this is for SPARK-11551. Please sign it off if looks good to you. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-9104][SPARK-9105][SPARK-9106][SPARK-910...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7753#issuecomment-163137831 **[Test build #47415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47415/consoleFull)** for PR 7753 at commit [`4b3dbe4`](https://gith

[GitHub] spark pull request: [SPARK-11551][DOC] Replace example code in ml-...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10219#issuecomment-163137574 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-11551][DOC] Replace example code in ml-...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10219#issuecomment-163137570 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 projec

[GitHub] spark pull request: [SPARK-11551][DOC] Replace example code in ml-...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10219#issuecomment-163137492 **[Test build #47413 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47413/consoleFull)** for PR 10219 at commit [`8748a88`](https://g

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9659 --- 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-12235][SPARKR] Enhance mutate() to supp...

2015-12-08 Thread sun-rui
GitHub user sun-rui opened a pull request: https://github.com/apache/spark/pull/10220 [SPARK-12235][SPARKR] Enhance mutate() to support replace existing columns. Make the behavior of mutate more consistent with that in dplyr, besides support for replacing existing columns. 1. Th

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163136743 Thanks for working on this! Merging to master. (Actually I thought I've already merged this one.) --- If your project is set up for it, you can reply to this email an

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/9659#discussion_r47057211 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala --- @@ -50,27 +50,33 @@ class ParquetFilterSu

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163136211 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163136210 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 projec

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163136125 **[Test build #47411 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47411/consoleFull)** for PR 10217 at commit [`5c71cd3`](https://g

[GitHub] spark pull request: [SPARK-12069][SQL] Update documentation with D...

2015-12-08 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/10060#discussion_r47056802 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/Encoder.scala --- @@ -19,20 +19,60 @@ package org.apache.spark.sql import java.lang.re

[GitHub] spark pull request: [SPARK-11551][DOC][Example]Replace example cod...

2015-12-08 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10002#issuecomment-163135345 @yinxusen Well, I'm probably too ignorant to review #10219 :) I'd ping @mengxr to sign it off. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-11551][DOC] Replace example code in ml-...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10219#issuecomment-163135226 **[Test build #47413 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47413/consoleFull)** for PR 10219 at commit [`8748a88`](https://gi

[GitHub] spark pull request: [SPARK-11551][DOC][Example]Replace example cod...

2015-12-08 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/10002#issuecomment-163133717 @somideshmukh Never mind, I add a new one on behalf of you: https://github.com/apache/spark/pull/10219 @liancheng if it looks good to you, please help me merge it afte

[GitHub] spark pull request: [SPARK-11551][DOC] Replace example code in ml-...

2015-12-08 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/10219#issuecomment-163133018 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 feature e

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix ```subset``` functi...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10217#issuecomment-163132940 **[Test build #47411 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47411/consoleFull)** for PR 10217 at commit [`5c71cd3`](https://gi

[GitHub] spark pull request: [SPARK-11551][DOC] Replace example code in ml-...

2015-12-08 Thread yinxusen
GitHub user yinxusen opened a pull request: https://github.com/apache/spark/pull/10219 [SPARK-11551][DOC] Replace example code in ml-features.md using include_example PR on behalf of @somideshmukh, thanks! You can merge this pull request into a Git repository by running: $ git

[GitHub] spark pull request: [SPARK-12227][SQL] Support drop multiple colum...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10218#issuecomment-163132221 **[Test build #47412 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47412/consoleFull)** for PR 10218 at commit [`2223526`](https://gi

[GitHub] spark pull request: [SPARK-12227][SQL] Support drop multiple colum...

2015-12-08 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/10218 [SPARK-12227][SQL] Support drop multiple columns specified by Column class in DataFrame API https://issues.apache.org/jira/browse/SPARK-12227 In this PR, I added the support to drop mu

[GitHub] spark pull request: [SPARK-11551][DOC][Example]Replace example cod...

2015-12-08 Thread somideshmukh
Github user somideshmukh commented on the pull request: https://github.com/apache/spark/pull/10002#issuecomment-163128740 Please let me know what needs to be done,I have not worked on python code. --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] spark pull request: [SPARK-11551][DOC][Example]Replace example cod...

2015-12-08 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/10002#issuecomment-163127889 Sure, I will do it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this f

[GitHub] spark pull request: [SPARK-12234] [SparkR] Fix subset error when o...

2015-12-08 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/10217 [SPARK-12234] [SparkR] Fix subset error when only set select argument Fix subset error when only set ```select``` argument. You can merge this pull request into a Git repository by running:

[GitHub] spark pull request: [SPARK-11551][DOC][Example]Replace example cod...

2015-12-08 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10002#issuecomment-163126528 @yinxusen Maybe you can fork this PR branch and get it merged. You may add a note in your PR description to remind committers to attribute your PR to @somideshmukh. W

[GitHub] spark pull request: [SPARK-11551][DOC][Example]Replace example cod...

2015-12-08 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/10002#issuecomment-163123743 @somideshmukh Do you still have time work on this? Pleas let me know ASAP thanks. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] spark pull request: [SPARK-12145][SQL] Command 'Set Role [ADMIN|NO...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10144#issuecomment-163122908 **[Test build #47410 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47410/consoleFull)** for PR 10144 at commit [`d3498f9`](https://gi

[GitHub] spark pull request: [SPARK-12161][SQL] Ignore order of predicates ...

2015-12-08 Thread windscope
Github user windscope commented on the pull request: https://github.com/apache/spark/pull/10163#issuecomment-163121275 To improve semanticEquals, we tried to implement a template function `Expression.splitWithCommutativeOperator[T: Manifest](): Seq[Expression]` so that we don't need t

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163121259 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163121258 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-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163121172 **[Test build #47407 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47407/consoleFull)** for PR 9659 at commit [`372c99e`](https://git

[GitHub] spark pull request: [SPARK-12164] [SQL] Decode the encoded values ...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10215#issuecomment-163120392 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 projec

[GitHub] spark pull request: [SPARK-12164] [SQL] Decode the encoded values ...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10215#issuecomment-163120394 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12164] [SQL] Decode the encoded values ...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10215#issuecomment-163120309 **[Test build #47405 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47405/consoleFull)** for PR 10215 at commit [`1e1ad19`](https://g

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

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8880#issuecomment-163119260 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

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

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8880#issuecomment-163119208 **[Test build #47409 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47409/consoleFull)** for PR 8880 at commit [`22c314e`](https://git

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

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8880#issuecomment-163119258 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-5682][Core] Add encrypted shuffle in sp...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8880#issuecomment-163117349 **[Test build #47409 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47409/consoleFull)** for PR 8880 at commit [`22c314e`](https://gith

[GitHub] spark pull request: [SPARK-12222] [Core] Deserialize RoaringBitmap...

2015-12-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10213 --- 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 ena

[GitHub] spark pull request: [SPARK-12222] [Core] Deserialize RoaringBitmap...

2015-12-08 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10213#issuecomment-163115766 LGTM, merging into master and 1.6, 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 projec

[GitHub] spark pull request: [SPARK-11016] Move RoaringBitmap to explicit K...

2015-12-08 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/9748#issuecomment-163115478 Sending to master should be enough, will be pickled into 1.6. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] spark pull request: [SPARK-12204][SPARKR] Implement drop method fo...

2015-12-08 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/10201#discussion_r47052792 --- Diff: R/pkg/R/DataFrame.R --- @@ -2223,3 +2215,43 @@ setMethod("with", newEnv <- assignNewEnv(data) eval(substitute(

[GitHub] spark pull request: [SPARK-12204][SPARKR] Implement drop method fo...

2015-12-08 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/10201#discussion_r47052771 --- Diff: R/pkg/R/generics.R --- @@ -421,6 +421,10 @@ setGeneric("corr", function(x, ...) {standardGeneric("corr") }) #' @export setGeneric("de

[GitHub] spark pull request: [SPARK-10780] [ML] Set initialModel in KMeans ...

2015-12-08 Thread jayantshekhar
Github user jayantshekhar commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-163110967 Still pondering over it and would luv to get inputs here. Do we create a Param of Vector Array and use it as the initialModel? clusterCenters in org.apache

[GitHub] spark pull request: [SPARK-12198] [SparkR] SparkR support read.par...

2015-12-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/10191#issuecomment-163110182 looks good. --- 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 featu

[GitHub] spark pull request: [SPARK-12198] [SparkR] SparkR support read.par...

2015-12-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/10191#issuecomment-163109528 sorry that's unrelated to this PR. Opened SPARK-12224 and SPARK-12232 --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] spark pull request: [SPARK-12181] Check Cached unaligned-access ca...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10181#issuecomment-163109335 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12181] Check Cached unaligned-access ca...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10181#issuecomment-163109332 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 projec

[GitHub] spark pull request: [SPARK-12181] Check Cached unaligned-access ca...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10181#issuecomment-163109273 **[Test build #47402 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47402/consoleFull)** for PR 10181 at commit [`16db32f`](https://g

[GitHub] spark pull request: [SPARK-10509][PYSPARK] Reduce excessive param ...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10216#issuecomment-163108450 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-10509][PYSPARK] Reduce excessive param ...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10216#issuecomment-163108449 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 projec

[GitHub] spark pull request: [SPARK-10509][PYSPARK] Reduce excessive param ...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10216#issuecomment-163108369 **[Test build #47408 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47408/consoleFull)** for PR 10216 at commit [`a976b78`](https://g

[GitHub] spark pull request: [SPARK-12188] [SQL] [FOLLOW-UP] Code refactori...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10214#issuecomment-163108214 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 projec

[GitHub] spark pull request: [SPARK-12188] [SQL] [FOLLOW-UP] Code refactori...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10214#issuecomment-163108215 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12188] [SQL] [FOLLOW-UP] Code refactori...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10214#issuecomment-163108147 **[Test build #47400 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47400/consoleFull)** for PR 10214 at commit [`145cd5b`](https://g

[GitHub] spark pull request: [SPARK-12228] [SQL] Try to run execution hive'...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10204#issuecomment-163108014 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12228] [SQL] Try to run execution hive'...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10204#issuecomment-163108011 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 projec

[GitHub] spark pull request: [SPARK-12228] [SQL] Try to run execution hive'...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10204#issuecomment-163107948 **[Test build #47404 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47404/consoleFull)** for PR 10204 at commit [`8546b8b`](https://g

[GitHub] spark pull request: [RFC] Add type-safe config keys to SparkConf.

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10205#issuecomment-163107724 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [RFC] Add type-safe config keys to SparkConf.

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10205#issuecomment-163107723 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 projec

[GitHub] spark pull request: [RFC] Add type-safe config keys to SparkConf.

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10205#issuecomment-163107657 **[Test build #47388 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47388/consoleFull)** for PR 10205 at commit [`bda7e2b`](https://g

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163107457 **[Test build #47407 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47407/consoleFull)** for PR 9659 at commit [`372c99e`](https://gith

[GitHub] spark pull request: [SPARK-12222] [Core] Deserialize RoaringBitmap...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10213#issuecomment-163107229 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 projec

[GitHub] spark pull request: [SPARK-12222] [Core] Deserialize RoaringBitmap...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10213#issuecomment-163107230 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12222] [Core] Deserialize RoaringBitmap...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10213#issuecomment-163107186 **[Test build #47396 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47396/consoleFull)** for PR 10213 at commit [`01d37b0`](https://g

[GitHub] spark pull request: [SPARK-10509][PYSPARK] Reduce excessive param ...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10216#issuecomment-163107144 **[Test build #47408 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47408/consoleFull)** for PR 10216 at commit [`a976b78`](https://gi

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163107048 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163107036 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163107047 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-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163107035 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-10299][ML] word2vec should allow users ...

2015-12-08 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/8513#issuecomment-163107088 @jkbradley addressed the issues (also cleaned up the rest of the tests in the same file) --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-11676][SQL] Parquet filter tests all pa...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9659#issuecomment-163106968 **[Test build #47391 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47391/consoleFull)** for PR 9659 at commit [`9861d96`](https://git

[GitHub] spark pull request: [SPARK-10509][PYSPARK] Reduce excessive param ...

2015-12-08 Thread holdenk
GitHub user holdenk opened a pull request: https://github.com/apache/spark/pull/10216 [SPARK-10509][PYSPARK] Reduce excessive param boiler plate code The current python ml params require cut-and-pasting the param setup and description between the class & ```__init__``` methods. Remo

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163106473 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-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163106474 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163106415 **[Test build #47395 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47395/consoleFull)** for PR 9687 at commit [`18c0326`](https://git

[GitHub] spark pull request: [SPARK-12164] [SQL] Decode the encoded values ...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10215#issuecomment-163106002 **[Test build #47405 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47405/consoleFull)** for PR 10215 at commit [`1e1ad19`](https://gi

  1   2   3   4   5   6   7   8   >