Github user squito commented on the issue:
https://github.com/apache/spark/pull/21212
can you add a test in MapOutputTrackerSuite and update the pr description
to include all the changes? but overall looks good.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21214#discussion_r185834096
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameRangeSuite.scala ---
@@ -153,23 +153,17 @@ class DataFrameRangeSuite extends QueryTest with
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21214
> Namely the test which killed threads passed and right after the upcoming
test stuck in submitting a job.
I think that indicates that there might be an issue with the code that
cancels
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20261
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20261
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90134/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20261
**[Test build #90134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90134/testReport)**
for PR 20261 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21212
If you have a heap dump, there are tools that can check for wasted space in
ArrayBuffer. Eg. [jxray](http://www.jxray.com/) or [YourKit Memory
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2876/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90130/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21163
**[Test build #90130 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90130/testReport)**
for PR 21163 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90139 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90139/testReport)**
for PR 21193 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21106
@hvanhovell If you don't have other comment, I will follow @cloud-fan's
suggestion to add a sql conf for it. Thanks.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90118/
Test PASSed.
---
Github user ala closed the pull request at:
https://github.com/apache/spark/pull/21227
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21021
**[Test build #90118 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90118/testReport)**
for PR 21021 at commit
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21182#discussion_r185827257
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -136,4 +138,6 @@ private[sql] class JSONOptions(
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/21227
LGTM - merging to 2.3. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user mn-mikke commented on the issue:
https://github.com/apache/spark/pull/21215
@maropu Really nice idea to create typed empty arrays via an `Literal`
expression! On the other hand, I feel that the end user shouldn't work with
classes from Catalyst internals if we consider
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21227
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90119/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21227
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21227
**[Test build #90119 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90119/testReport)**
for PR 21227 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90138/testReport)**
for PR 21217 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/21225#discussion_r185821003
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/window/WindowExec.scala
---
@@ -197,6 +198,7 @@ case class WindowExec(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21173
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2875/
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21182#discussion_r185820210
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -136,4 +138,6 @@ private[sql] class JSONOptions(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21173
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90135 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90135/testReport)**
for PR 21193 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90135/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21215
**[Test build #90137 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90137/testReport)**
for PR 21215 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21217
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2874/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21173
**[Test build #90136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90136/testReport)**
for PR 21173 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90135 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90135/testReport)**
for PR 21193 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21225
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21193
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90120/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21226
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90114/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21226
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90117/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90120 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90120/testReport)**
for PR 21193 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20261
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2873/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20261
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21028
**[Test build #90117 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90117/testReport)**
for PR 21028 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21226
**[Test build #90114 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90114/testReport)**
for PR 21226 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2872/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21229
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2871/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21229
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20261
**[Test build #90134 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90134/testReport)**
for PR 20261 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2870/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/20261
Jenkins, test this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2869/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21229
**[Test build #90132 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90132/testReport)**
for PR 21229 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #90133 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90133/testReport)**
for PR 21129 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #90131 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90131/testReport)**
for PR 21129 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90131/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2868/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21229
cc @jiangxb1987 @rxin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21214
**[Test build #90129 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90129/testReport)**
for PR 21214 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #90131 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90131/testReport)**
for PR 21129 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21163
**[Test build #90130 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90130/testReport)**
for PR 21163 at commit
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/21229
[SPARK-23697][CORE] LegacyAccumulatorWrapper should define isZero correctly
## What changes were proposed in this pull request?
It's possible that Accumulators of Spark 1.x may no longer
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21212
@squito @cloud-fan @jiangxb1987
Thanks a lot for review.
> shall we also optimize the space usage for MapStatus
@cloud-fan do you mean optimize space usage for MapStatus when
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21212
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2867/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21212
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21028
**[Test build #90128 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90128/testReport)**
for PR 21028 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21212
**[Test build #90127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90127/testReport)**
for PR 21212 at commit
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21227#discussion_r185801594
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/WritableColumnVector.java
---
@@ -81,7 +81,9 @@ public void close() {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21225
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21225
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90113/
Test PASSed.
---
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185800917
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/regression.scala
---
@@ -0,0 +1,190 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2866/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21225
**[Test build #90113 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90113/testReport)**
for PR 21225 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #90126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90126/testReport)**
for PR 21129 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90126/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #90126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90126/testReport)**
for PR 21129 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185794875
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -112,6 +112,102 @@ object JavaCode {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21068
**[Test build #90125 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90125/testReport)**
for PR 21068 at commit
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/21214
> Just curious how this happened?
I've taken a look at 2 logs you've shown and seen the same pattern. Namely
the test which killed threads passed and right after the upcoming test stuck in
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185794699
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -120,13 +216,15 @@ object JavaCode {
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21225#discussion_r185793645
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/window/WindowExec.scala
---
@@ -114,7 +114,8 @@ case class WindowExec(
*
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185791203
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/regression.scala
---
@@ -0,0 +1,190 @@
+/*
+ * Licensed
Github user juliuszsompolski commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185792840
--- Diff: R/pkg/R/functions.R ---
@@ -3184,6 +3191,7 @@ setMethod("create_map",
#' collect(select(df2, collect_list(df2$gear)))
#'
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185792266
--- Diff: R/pkg/R/functions.R ---
@@ -963,6 +964,7 @@ setMethod("kurtosis",
#' last(df$c, TRUE)
#' }
#' @note last since 1.4.0
+#'
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/21214#discussion_r185792166
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameRangeSuite.scala ---
@@ -153,23 +153,17 @@ class DataFrameRangeSuite extends
Github user juliuszsompolski commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185791719
--- Diff: R/pkg/R/functions.R ---
@@ -963,6 +964,7 @@ setMethod("kurtosis",
#' last(df$c, TRUE)
#' }
#' @note last since 1.4.0
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185791498
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
401 - 500 of 724 matches
Mail list logo