Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11267
---
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 user shivaram commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-195134060
Thanks @yanboliang -- Sorry for the delay in looking at this. I think this
looks good. Merging to master
---
If your project is set up for it, you can reply to this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-187001699
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-187001694
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-187001455
**[Test build #51656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51656/consoleFull)**
for PR 11267 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186995268
**[Test build #51656 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51656/consoleFull)**
for PR 11267 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/11267#discussion_r53583948
--- Diff: R/pkg/R/functions.R ---
@@ -536,15 +536,27 @@ setMethod("factorial",
#'
#' Aggregate function: returns the first value in a group.
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186993289
@shivaram Several R packages have similar function such as
[first](http://www.inside-r.org/packages/cran/pastecs/docs/first),
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/11267#discussion_r53573356
--- Diff: R/pkg/R/functions.R ---
@@ -536,15 +536,27 @@ setMethod("factorial",
#'
#' Aggregate function: returns the first value in a group.
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186380904
Is this option used by base R / other R libraries like dplyr ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186134049
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186134043
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186133714
**[Test build #51546 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51546/consoleFull)**
for PR 11267 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11267#issuecomment-186124658
**[Test build #51546 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51546/consoleFull)**
for PR 11267 at commit
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/11267
[SPARK-13389] [SparkR] SparkR support first/last with ignore NAs
## What changes were proposed in this pull request?
SparkR support first/last with ignore NAs
cc @sun-rui
15 matches
Mail list logo
