[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[sun-rui]

Github user sun-rui commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-151055969
  
@shivaram, [R lag 
function](https://stat.ethz.ch/R-manual/R-devel/library/stats/html/lag.html) is 
for time series objects, which is irrelevant to the lead/lag here in SQL window 
functions.

There are some functions in dplyr package mimicing SQL window functions, 
including lead and lag. Refer to 
https://cran.r-project.org/web/packages/dplyr/vignettes/window-functions.html 
and https://cran.r-project.org/web/packages/dplyr/dplyr.pdf. But a quick glance 
at these functions shows that they are working on vectors (for example, lag 
right shifts a vector by offset), so the semantic of them are different from 
those in Spark.

So for now I won't change this 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 your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[sun-rui]

Github user sun-rui commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-151350204
  
Yes. the lag will be masked. Just as discussed before, sometimes, this is 
allowed, as I assume lag is not so commonly and frequently used ("dplyr" masks 
lag also). Users can still use stats::lag if they do want to use 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[sun-rui]

Github user sun-rui commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-151375443
  
@shivaram, for your suggestion that "we should a make a list of functions 
that we mask and are incompatible", I submitted 
https://issues.apache.org/jira/browse/SPARK-11339


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[shivaram]

Github user shivaram commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-151366290
  
Hmm I see. In the case of `lag` I can see your point. But at a high level I 
think we should a make a list of functions that we mask and are incompatible 
(i.e. things like `head` are not a problem) 

Otherwise LGTM. Merging this.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/9193


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[shivaram]

Github user shivaram commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-151195921
  
Ok. The thing I was worried about is that if somebody loads SparkR they 
won't be able to use `lag` from stats -- Can you check if that is the case ?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[shivaram]

Github user shivaram commented on a diff in the pull request:

https://github.com/apache/spark/pull/9193#discussion_r42886920
  
--- Diff: R/pkg/R/functions.R ---
@@ -2008,3 +2008,101 @@ setMethod("ifelse",
 "otherwise", no)
   column(jc)
   })
+
+## Window functions##
+
+#' cumeDist
+#'
+#' Window function: returns the cumulative distribution of values within a 
window partition,
+#' i.e. the fraction of rows that are below the current row.
+#' 
+#'   N = total number of rows in the partition
+#'   cumeDist(x) = number of values before (and including) x / N
+#'   
+#' This is equivalent to the CUME_DIST function in SQL.
+#'
+#' @rdname cumeDist
+#' @name cumeDist
+#' @family window_funcs
+#' @export
+#' @examples \dontrun{cumeDist()}
+setMethod("cumeDist",
+  signature(x = "missing"),
+  function() {
+jc <- callJStatic("org.apache.spark.sql.functions", "cumeDist")
+column(jc)
+  })
+
+#' lag
+#'
+#' Window function: returns the value that is `offset` rows before the 
current row, and
+#' `defaultValue` if there is less than `offset` rows before the current 
row. For example,
+#' an `offset` of one will return the previous row at any given point in 
the window partition.
+#' 
+#' This is equivalent to the LAG function in SQL.
+#'
+#' @rdname lag
+#' @name lag
+#' @family window_funcs
+#' @export
+#' @examples \dontrun{lag(df$c)}
+setMethod("lag",
--- End diff --

There is a method called `lag` in base R that this would conflict with. 
Could we try to use the same argument names as that function ?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150124146
  
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150124071
  
**[Test build #44128 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44128/consoleFull)**
 for PR 9193 at commit 
[`50a0d27`](https://github.com/apache/spark/commit/50a0d2744d0233676683527120405d3aa4e42b37).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150124147
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44128/
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150105205
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150105217
  
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150105522
  
**[Test build #44128 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44128/consoleFull)**
 for PR 9193 at commit 
[`50a0d27`](https://github.com/apache/spark/commit/50a0d2744d0233676683527120405d3aa4e42b37).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[sun-rui]

Github user sun-rui commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-150105010
  
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149844454
  
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149844384
  
**[Test build #44051 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44051/consoleFull)**
 for PR 9193 at commit 
[`50a0d27`](https://github.com/apache/spark/commit/50a0d2744d0233676683527120405d3aa4e42b37).
 * This patch **fails Spark unit tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149844457
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44051/
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[felixcheung]

Github user felixcheung commented on a diff in the pull request:

https://github.com/apache/spark/pull/9193#discussion_r42587435
  
--- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala 
---
@@ -224,7 +224,8 @@ private[r] class RBackendHandler(server: RBackend)
 case _ => parameterType
   }
 }
-if (!parameterWrapperType.isInstance(args(i))) {
+if (!(!parameterType.isPrimitive && args(i) == null) &&
+!parameterWrapperType.isInstance(args(i))) {
--- End diff --

nit: this is kinda hard to read with the double negatives. consider:
```
if ((parameterType.isPrimitive || args(i) != null) &&
!parameterWrapperType.isInstance(args(i))) {
```
?



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149793886
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149793899
  
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149795220
  
**[Test build #44050 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44050/consoleFull)**
 for PR 9193 at commit 
[`0504092`](https://github.com/apache/spark/commit/05040924e5ea6be9038a36a5299c5da2d4e5ca4d).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[sun-rui]

Github user sun-rui commented on a diff in the pull request:

https://github.com/apache/spark/pull/9193#discussion_r42595273
  
--- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala 
---
@@ -224,7 +224,8 @@ private[r] class RBackendHandler(server: RBackend)
 case _ => parameterType
   }
 }
-if (!parameterWrapperType.isInstance(args(i))) {
+if (!(!parameterType.isPrimitive && args(i) == null) &&
+!parameterWrapperType.isInstance(args(i))) {
--- End diff --

OK. done.

Anyway, I feel painful that Scala does not support 'continue' statement. So 
the code here still not so 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149813584
  
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149813507
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149815360
  
**[Test build #44051 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44051/consoleFull)**
 for PR 9193 at commit 
[`50a0d27`](https://github.com/apache/spark/commit/50a0d2744d0233676683527120405d3aa4e42b37).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149815518
  
**[Test build #44050 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44050/consoleFull)**
 for PR 9193 at commit 
[`0504092`](https://github.com/apache/spark/commit/05040924e5ea6be9038a36a5299c5da2d4e5ca4d).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149815640
  
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149815642
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44050/
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[sun-rui]

GitHub user sun-rui opened a pull request:

https://github.com/apache/spark/pull/9193

[SPARK-11209][SPARKR] Add window functions into SparkR [step 1].



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/sun-rui/spark SPARK-11209

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/9193.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #9193


commit 21e098c02f64dbf9dd2e5af3851ccda8247fc2e4
Author: Sun Rui 
Date:   2015-10-20T12:36:20Z

[SPARK-11209][SPARKR] Add window functions into SparkR [step 1].

commit 4fedfcb67668294780484ef4a47204b53e6f9ea2
Author: Sun Rui 
Date:   2015-10-21T05:00:59Z

Improve RBackend to allow for passing null to parameter of object types.

commit ed57eb8f4c88fe5e956d48ea3ce47804c05a19d9
Author: Sun Rui 
Date:   2015-10-21T05:01:16Z

Add test cases.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149784768
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44043/
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149784764
  
**[Test build #44043 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44043/consoleFull)**
 for PR 9193 at commit 
[`ed57eb8`](https://github.com/apache/spark/commit/ed57eb8f4c88fe5e956d48ea3ce47804c05a19d9).
 * This patch **fails R style tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149784767
  
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149783538
  
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149783525
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11209][SPARKR] Add window functions int...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9193#issuecomment-149784461
  
**[Test build #44043 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44043/consoleFull)**
 for PR 9193 at commit 
[`ed57eb8`](https://github.com/apache/spark/commit/ed57eb8f4c88fe5e956d48ea3ce47804c05a19d9).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org