Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16344#discussion_r93289668
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -592,6 +629,59 @@ object
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16356
[SPARK-18949] [SQL] Add repairTable API to Catalog
### What changes were proposed in this pull request?
Currently, we only have a SQL interface for recovering all the partitions
in the
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16356
We have two SQL equivalent commands:
- ALTER TABLE table RECOVER PARTITIONS;
- MSCK REPAIR TABLE table;
I am not good at naming. How about `recoverPartitions`?
---
If your
GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/16357
[SPARK-18928][branch-2.0]Check TaskContext.isInterrupted() in FileScanRDD,
JDBCRDD & UnsafeSorter
This is a branch-2.0 backport of #16340; the original description follows:
## What
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16337
yea those sound great.
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/16337
If the uniqueness is a problem, we can also just update SQLQueryTestSuite
to use the path rather than just file name.
---
If your project is set up for it, you can reply to this email and have your
GitHub user imatiach-msft opened a pull request:
https://github.com/apache/spark/pull/16355
[SPARK-16473][MLLIB] Fix BisectingKMeans Algorithm failing in edge case
[SPARK-16473][MLLIB] Fix BisectingKMeans Algorithm failing in edge case
where no children exist in updateAssignments
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16352#discussion_r93300223
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -747,7 +747,7 @@ class SQLContext private[sql](val sparkSession:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16336
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 rxin commented on the issue:
https://github.com/apache/spark/pull/16356
What is the SQL equivalent command? MSCK? Should we match that?
---
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
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16344#discussion_r93290858
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -592,6 +629,59 @@ object
Github user merlintang commented on the issue:
https://github.com/apache/spark/pull/15819
@cloud-fan @gatorsmile I have backport the code from #16134, can you
verify and backport this to spark 1.6.x?
---
If your project is set up for it, you can reply to this email and have your
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15996
LGTM
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16336
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70415/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16336
**[Test build #70415 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70415/testReport)**
for PR 16336 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16356
**[Test build #70416 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70416/testReport)**
for PR 16356 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16356
Sure, will do. 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
enabled and
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16356
We should also add the Python API.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16354
**[Test build #70412 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70412/testReport)**
for PR 16354 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15819
Let me backport it to Spark 2.0.x at first. We need to follow the backport
ordering
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user nsyca commented on the issue:
https://github.com/apache/spark/pull/16337
We can live with the uniqueness at this point. I'd like to get your
feedback (positive, I hope) on the structure of the directories and the naming
convention of the test files so we can move forward
Github user davies commented on the issue:
https://github.com/apache/spark/pull/15980
lgtm
---
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
Github user merlintang commented on the issue:
https://github.com/apache/spark/pull/15819
@gatorsmile Great! thanks so much, because I was pinged multiple times for
this bug. :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/16357
LGTM
---
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
GitHub user squito opened a pull request:
https://github.com/apache/spark/pull/16354
[SPARK-18886][Scheduler][WIP] Adjust Delay scheduling to prevent
under-utilization of cluster
## What changes were proposed in this pull request?
This is a significant change to delay
Github user nsyca commented on the issue:
https://github.com/apache/spark/pull/16337
I have tested a few runs on `SQLQueryTestSuite` to confirm it allows to
have sub-directories under
`sql/core/src/test/resources/sql-tests/[inputs|results]` to group test files
further. By reading
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16344#discussion_r93291042
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -592,6 +629,59 @@ object GeneralizedLinearRegression
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16344#discussion_r93290854
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -242,7 +275,7 @@ class GeneralizedLinearRegression
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16356
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70416/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16357
**[Test build #70417 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70417/consoleFull)**
for PR 16357 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16356
Yea recoverPartitions sound a lot better.
---
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/16356
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/16356
**[Test build #70416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70416/testReport)**
for PR 16356 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16343
**[Test build #70413 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70413/testReport)**
for PR 16343 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/16322#discussion_r93296609
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -282,7 +289,7 @@ class StreamExecution(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16355
Can one of the admins verify this patch?
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16343
**[Test build #70414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70414/testReport)**
for PR 16343 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/16354
@mridulm @markhamstra @kayousterhout
This is *not* ready to merge -- it needs some cleanup and more tests -- but
I thought that seeing an implementation might help think through the design. I
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16354#discussion_r93292503
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala ---
@@ -549,11 +546,15 @@ class TaskSetManagerSuite extends SparkFunSuite
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15996#discussion_r93298271
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -364,48 +365,149 @@ final class DataFrameWriter[T] private[sql](ds:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16336
Thank you, @srowen ! I will update the `README.md` , too.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16336
**[Test build #70415 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70415/testReport)**
for PR 16336 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16356
cc @rxin @cloud-fan @ericl
---
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/16343
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/16304
**[Test build #70420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70420/testReport)**
for PR 16304 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70414/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
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/16322
**[Test build #70425 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70425/testReport)**
for PR 16322 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16322
@marmbrus updated as per our offline discussion. I also added a new method
`assertAwaitThread` to avoid the user calling the awaitXXX methods inside the
stream thread (e.g., in the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70417/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16357
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 brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/16361#discussion_r93347240
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -48,28 +48,30 @@ class
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16357#discussion_r93321229
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeInMemorySorter.java
---
@@ -256,6 +259,14 @@ public boolean hasNext() {
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/16357#discussion_r93323151
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala
---
@@ -88,7 +88,15 @@ class FileScanRDD(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16356
**[Test build #70423 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70423/testReport)**
for PR 16356 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16359
**[Test build #70424 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70424/testReport)**
for PR 16359 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16358
**[Test build #70419 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70419/consoleFull)**
for PR 16358 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16361
**[Test build #70432 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70432/testReport)**
for PR 16361 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16361#discussion_r93346662
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/RowBasedHashMapGenerator.scala
---
@@ -43,28 +43,30 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16361
**[Test build #70433 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70433/testReport)**
for PR 16361 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16253
Looks like others are not really interested in reviewing. I'll take another
closer look at the model tomorrow, just to make sure.
---
If your project is set up for it, you can reply to this email
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70413/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16354
**[Test build #70412 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70412/testReport)**
for PR 16354 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16356
**[Test build #70422 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70422/testReport)**
for PR 16356 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16356#discussion_r93329801
--- Diff: project/MimaExcludes.scala ---
@@ -37,7 +37,9 @@ object MimaExcludes {
// Exclude rules for 2.2.x
lazy val v22excludes =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16356
**[Test build #70418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70418/testReport)**
for PR 16356 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16360
**[Test build #70429 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70429/testReport)**
for PR 16360 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16358
**[Test build #70419 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70419/consoleFull)**
for PR 16358 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16263
---
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16356#discussion_r93335790
--- Diff: python/pyspark/sql/catalog.py ---
@@ -258,6 +258,11 @@ def refreshTable(self, tableName):
"""Invalidate and refresh all the cached
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/15666#discussion_r93338899
--- Diff: core/src/main/scala/org/apache/spark/TestUtils.scala ---
@@ -164,6 +164,27 @@ private[spark] object TestUtils {
GitHub user tdas opened a pull request:
https://github.com/apache/spark/pull/16360
Made update mode public
## What changes were proposed in this pull request?
Made update mode public. As part of that here are the changes.
- Update DatastreamWriter to accept "update"
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16211
Maybe @davies has some cycles now that 2.1 is out :)
---
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 issue:
https://github.com/apache/spark/pull/16356
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/16356
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70418/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16343
**[Test build #70431 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70431/testReport)**
for PR 16343 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16336
Thank you for review, @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
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16253
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/16356
**[Test build #70418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70418/testReport)**
for PR 16356 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16357#discussion_r93323712
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeInMemorySorter.java
---
@@ -256,6 +259,14 @@ public boolean hasNext() {
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16357#discussion_r93326959
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeInMemorySorter.java
---
@@ -256,6 +259,14 @@ public boolean hasNext() {
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16356
We can laso merge this in branch-2.1. So let's do 2.1.1 as since version.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16359#discussion_r93335284
--- Diff: pom.xml ---
@@ -179,7 +179,7 @@
4.5.3
1.1
2.52.0
-2.8
+2.6
--- End diff --
Yep, looks
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16356#discussion_r93337049
--- Diff: python/pyspark/sql/catalog.py ---
@@ -258,6 +258,11 @@ def refreshTable(self, tableName):
"""Invalidate and refresh all the cached
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16360
**[Test build #70428 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70428/testReport)**
for PR 16360 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
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/16343
**[Test build #70427 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70427/testReport)**
for PR 16343 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70427/
Test FAILed.
---
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16361#discussion_r93346634
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -48,28 +48,30 @@ class
Github user davies commented on the issue:
https://github.com/apache/spark/pull/16263
Merging this into master and 2.1 branch, 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
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/16282#discussion_r93331357
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -536,11 +570,12 @@ private[kafka010] object
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16343
**[Test build #70421 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70421/testReport)**
for PR 16343 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70421/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16343
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 holdenk commented on the issue:
https://github.com/apache/spark/pull/16336
I think for new Python or Scala developers there is a benefit in skipping
the R docs because they require some extra setup work to build and having the
process simplified would be good for
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16356
**[Test build #70426 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70426/testReport)**
for PR 16356 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16342
LGTM. Thanks! Merging to master and 2.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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16360
**[Test build #70430 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70430/testReport)**
for PR 16360 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16358
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 actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16344
@srowen Thanks for the comments. Really helpful. I have made a new commit
that addresses the issues you raised:
- I think the use of a global family object does not work well for the
1 - 100 of 429 matches
Mail list logo
