Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98838566
--- Diff: R/pkg/inst/tests/testthat/test_utils.R ---
@@ -17,6 +17,9 @@
context("functions in utils.R")
+# Ensure Spark is installed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16043
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72229/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16043
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16043
**[Test build #72229 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72229/testReport)**
for PR 16043 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16761
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16761
**[Test build #72233 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72233/testReport)**
for PR 16761 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16761
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72233/
Test FAILed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98837222
--- Diff: R/pkg/inst/tests/testthat/test_utils.R ---
@@ -17,6 +17,9 @@
context("functions in utils.R")
+# Ensure Spark is installed
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98837066
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -27,6 +27,9 @@ library(SparkR)
We use default settings in which it runs in local mode. It
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16736
BTW, you need to update the PR title and description.
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16736
Overall, looks good to me. cc @cloud-fan
---
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 user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16736#discussion_r98836889
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/ConfigBuilder.scala ---
@@ -90,6 +90,14 @@ private[spark] class TypedConfigBuilder[T](
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16758
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72230/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16758
**[Test build #72230 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72230/testReport)**
for PR 16758 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98836638
--- Diff: R/pkg/inst/tests/testthat/test_utils.R ---
@@ -17,6 +17,9 @@
context("functions in utils.R")
+# Ensure Spark is installed
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16736#discussion_r98836570
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala ---
@@ -221,6 +221,10 @@ class SQLConfSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16737
**[Test build #72238 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72238/testReport)**
for PR 16737 at commit
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/15415#discussion_r98836498
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -0,0 +1,251 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16737
I think `LIBSVM` and `TEXT` are the last two built-in sources that do not
support case sensitivity.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16737#discussion_r98836103
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/source/libsvm/LibSVMOptions.scala ---
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache Software
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16753
LGTM except two comments.
---
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
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/12135
@holdenk Updated! Thanks for your careful checking.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/12135
**[Test build #72237 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72237/testReport)**
for PR 12135 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16753#discussion_r98835277
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala
---
@@ -53,33 +53,31 @@ class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16753#discussion_r98835197
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala
---
@@ -53,33 +53,31 @@ class
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16751
Hi, @rxin .
Sure, I'll try to put them in a single PR except the ongoing one.
BTW, every time, I noticed that committers have a better and broader
perspective than me.
Do you have
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/16727
@holdenk I created another jira to track this issue. Thanks all for
revewing!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98834936
--- Diff: R/pkg/inst/tests/testthat/test_utils.R ---
@@ -17,6 +17,9 @@
context("functions in utils.R")
+# Ensure Spark is installed
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98834935
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -27,6 +27,9 @@ library(SparkR)
We use default settings in which it runs in local mode.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16690
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16690
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72228/
Test PASSed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16762#discussion_r98834480
--- Diff: sql/core/src/test/resources/sql-tests/inputs/cross-join.sql ---
@@ -33,3 +33,5 @@ create temporary view D(d, vd) as select * from nt1;
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16690
**[Test build #72228 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72228/testReport)**
for PR 16690 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16750
**[Test build #72236 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72236/testReport)**
for PR 16750 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16620
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72227/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16620
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
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16750#discussion_r98834088
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVOptions.scala
---
@@ -161,12 +163,3 @@ private[csv] class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16750#discussion_r98834068
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -329,7 +332,17 @@ class DataFrameReader private[sql](sparkSession:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16620
**[Test build #72227 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72227/testReport)**
for PR 16620 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16750#discussion_r98834049
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -297,7 +300,7 @@ def text(self, paths):
def csv(self, path, schema=None, sep=None, encoding=None,
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98833756
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -27,6 +27,9 @@ library(SparkR)
We use default settings in which it runs in local mode. It
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16720#discussion_r98833957
--- Diff: R/pkg/inst/tests/testthat/test_utils.R ---
@@ -17,6 +17,9 @@
context("functions in utils.R")
+# Ensure Spark is installed
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16762
**[Test build #72235 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72235/testReport)**
for PR 16762 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16762#discussion_r98833830
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -584,24 +602,37 @@ class JoinSuite extends QueryTest with
SharedSQLContext
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16762#discussion_r98833748
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -215,18 +215,36 @@ class JoinSuite extends QueryTest with
SharedSQLContext
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16578#discussion_r98833772
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/GetStructField2.scala
---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16578#discussion_r98833717
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/GetStructField2.scala
---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15279
What if we have a bunch of default values when arguments are not set, and
those are the values we could test with? This way the same sample code can run
with and without arguments?
---
If
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16762
[SPARK-19419] [SPARK-19420] Fix the cross join detection
### What changes were proposed in this pull request?
There are two issues in the existing detection of cartesian products.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16603
**[Test build #72234 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72234/testReport)**
for PR 16603 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16603
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,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16761
**[Test build #72233 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72233/testReport)**
for PR 16761 at commit
GitHub user wangmiao1981 opened a pull request:
https://github.com/apache/spark/pull/16761
[BackPort-2.1][SPARK-19319][SparkR]:SparkR Kmeans summary returns error
when the cluster size doesn't equal to k
## What changes were proposed in this pull request?
Backport fix of
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16739
**[Test build #72232 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72232/testReport)**
for PR 16739 at commit
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/1
I will backport it soon. 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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16620
**[Test build #72231 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72231/testReport)**
for PR 16620 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16722
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72225/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16722
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16722
**[Test build #72225 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72225/testReport)**
for PR 16722 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16740
@sethah Thanks for the clarification and providing an implementation. So,
the pros is some speed improvement and the cons is the increased complexity
(now we have three case: one for intercept
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16620
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16620
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72226/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16620
**[Test build #72226 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72226/testReport)**
for PR 16620 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/1
merged, thanks!
I think it'll good to have this in branch-2.1 - @wangmiao1981 would you by
any chance would like to backport this fix?
---
If your project is set up for it, you can reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16758
**[Test build #72230 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72230/testReport)**
for PR 16758 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/16740
I agree having a special case is unsatisfying from an engineering
perspective. In Spark it's a bit different than R since every iteration of IRLS
will launch a Spark job, making a pass over the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16043
**[Test build #72229 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72229/testReport)**
for PR 16043 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16725
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16725
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72224/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16725
**[Test build #72224 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72224/testReport)**
for PR 16725 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16690
**[Test build #72228 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72228/testReport)**
for PR 16690 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16620
**[Test build #72227 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72227/testReport)**
for PR 16620 at commit
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/16603
Thanks for the review @vanzin , I will leave this open for a day in case
someone else also wants to review; and will commit tomorrow.
---
If your project is set up for it, you can reply to this
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16740
@sethah Thanks for your input. I can add more tests, but they are not
adding too much since the algorithm is already tested in other tests.
The analytical approach does not integrate
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16603
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16603
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72221/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16603
**[Test build #72221 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72221/testReport)**
for PR 16603 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16620
**[Test build #72226 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72226/testReport)**
for PR 16620 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16722
**[Test build #72225 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72225/testReport)**
for PR 16722 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/16722
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72223/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16758
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16758
**[Test build #72223 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72223/testReport)**
for PR 16758 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16722
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72220/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16722
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16722
**[Test build #72220 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72220/testReport)**
for PR 16722 at commit
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16620#discussion_r98819685
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1212,8 +1223,9 @@ class DAGScheduler(
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/16578#discussion_r98819150
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/GetStructField2.scala
---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/16536
Thanks a bunch @holdenk
---
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
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16727
The original PR was merged a few days ago, would you be OK with making a
new JIRA just to avoid confusion? You can make it related to the previous JIRA.
But otherwise looks good to me.
---
If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16758
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72217/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16758
**[Test build #72217 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72217/testReport)**
for PR 16758 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16536
---
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 holdenk commented on the issue:
https://github.com/apache/spark/pull/16536
Going to go ahead and merge. Still need to sort out the JIRA permissions so
will take a bit for me to get that updated for you.
---
If your project is set up for it, you can reply to this email
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16725
LGTM pending tests
---
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,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16725
**[Test build #72224 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72224/testReport)**
for PR 16725 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16725
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 this feature
enabled and
1 - 100 of 399 matches
Mail list logo
