Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132440833
I open a PR for `ifelse`.
https://issues.apache.org/jira/browse/SPARK-10106
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132440315
Sure. Thank you for merging 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8266
---
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 user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132431814
LGTM. Merging this. @yu-iskw Can you open a JIRA for the `if-else` issue ?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132429348
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132429349
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132429295
[Test build #41198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41198/console)
for PR 8266 at commit
[`8a1fa07`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132427697
[Test build #41198 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41198/consoleFull)
for PR 8266 at commit
[`8a1fa07`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132426952
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132426940
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 user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132426562
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 user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37374083
--- Diff: R/pkg/R/functions.R ---
@@ -163,3 +163,17 @@ setMethod("n", signature(x = "Column"),
function(x) {
count(x)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132414569
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132414504
[Test build #41190 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41190/console)
for PR 8266 at commit
[`3280eeb`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132414568
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 user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37372853
--- Diff: R/pkg/R/functions.R ---
@@ -163,3 +163,17 @@ setMethod("n", signature(x = "Column"),
function(x) {
count(x)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132411260
[Test build #41190 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41190/consoleFull)
for PR 8266 at commit
[`3280eeb`](https://gith
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37372413
--- Diff: R/pkg/R/functions.R ---
@@ -163,3 +163,17 @@ setMethod("n", signature(x = "Column"),
function(x) {
count(x)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132410324
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132410336
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 user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132410178
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132409540
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132409541
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132407477
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132407461
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 user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132407292
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 user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132407271
@shivaram could you review it?
- Remove `when` in `column.R` and `%otherwise%` infix operator
- Update `NAMESPACE`
- Rebase on the master branch
---
If
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37370608
--- Diff: R/pkg/R/column.R ---
@@ -203,3 +203,49 @@ setMethod("%in%",
jc <- callJMethod(x@jc, "in", table)
return(column(jc))
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37370414
--- Diff: R/pkg/R/column.R ---
@@ -203,3 +203,49 @@ setMethod("%in%",
jc <- callJMethod(x@jc, "in", table)
return(column(jc)
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37370384
--- Diff: R/pkg/R/functions.R ---
@@ -163,3 +163,18 @@ setMethod("n", signature(x = "Column"),
function(x) {
count(x)
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37370211
--- Diff: R/pkg/R/column.R ---
@@ -203,3 +203,49 @@ setMethod("%in%",
jc <- callJMethod(x@jc, "in", table)
return(column(jc))
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37322106
--- Diff: R/pkg/R/column.R ---
@@ -203,3 +203,49 @@ setMethod("%in%",
jc <- callJMethod(x@jc, "in", table)
return(column(jc)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132270515
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132270513
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132270357
[Test build #41139 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41139/console)
for PR 8266 at commit
[`4346e62`](https://github.
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8266#discussion_r37321436
--- Diff: R/pkg/R/functions.R ---
@@ -163,3 +163,18 @@ setMethod("n", signature(x = "Column"),
function(x) {
count(x)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132263520
[Test build #41139 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41139/consoleFull)
for PR 8266 at commit
[`4346e62`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132262018
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132262035
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 user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132122504
@shivaram could you review 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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132122118
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132122120
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132122004
[Test build #4 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/4/console)
for PR 8266 at commit
[`b7ee3f8`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132119262
[Test build #4 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/4/consoleFull)
for PR 8266 at commit
[`b7ee3f8`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132118418
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132118414
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132118293
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132118312
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132118301
[Test build #41109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41109/console)
for PR 8266 at commit
[`7beacea`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132113941
[Test build #41109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41109/consoleFull)
for PR 8266 at commit
[`7beacea`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132112977
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132112919
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132104378
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132104372
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132103910
[Test build #41098 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41098/console)
for PR 8266 at commit
[`f80f7c0`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132098784
[Test build #41098 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41098/consoleFull)
for PR 8266 at commit
[`f80f7c0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132097857
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8266#issuecomment-132097818
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 user yu-iskw opened a pull request:
https://github.com/apache/spark/pull/8266
[SPARK-10075][SparkR] Add `when` expressino function in SparkR
- Add `when` and `otherwise` as `Column` methods
- Add `When` as an expression function
- Add `%otherwise%` infix as an alias of
59 matches
Mail list logo