Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20750
**[Test build #88017 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88017/testReport)**
for PR 20750 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20744
**[Test build #88018 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88018/testReport)**
for PR 20744 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20744
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88018/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20744
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/20295
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/1328/
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172616601
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamWriter.java
---
@@ -27,6 +28,9 @@
*
* Streaming queries
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172616831
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2.scala
---
@@ -54,7 +55,14 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20742
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/1330/
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172618681
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamingDataWriterFactory.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20742
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/20686
**[Test build #88023 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88023/testReport)**
for PR 20686 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20751
**[Test build #88022 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88022/testReport)**
for PR 20751 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20744
**[Test build #88018 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88018/testReport)**
for PR 20744 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172613993
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamingDataWriterFactory.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20295
**[Test build #88020 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88020/testReport)**
for PR 20295 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20295
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/20750
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88009/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20751
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/1329/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20751
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20744
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20744#discussion_r172607077
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -148,14 +148,14 @@ class HistoryServer(
appId: String,
Github user zhouyejoe commented on a diff in the pull request:
https://github.com/apache/spark/pull/20744#discussion_r172609731
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -148,14 +148,14 @@ class HistoryServer(
appId:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20742
**[Test build #88021 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88021/testReport)**
for PR 20742 at commit
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/20686#discussion_r172616958
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/QuantileDiscretizerSuite.scala
---
@@ -324,19 +352,46 @@ class QuantileDiscretizerSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88010/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20751
**[Test build #88010 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88010/testReport)**
for PR 20751 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20751
**[Test build #88011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88011/testReport)**
for PR 20751 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20750#discussion_r172634434
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Projection.scala
---
@@ -146,6 +145,18 @@ object UnsafeProjection {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20752
**[Test build #88019 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88019/testReport)**
for PR 20752 at commit
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172624233
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamingDataWriterFactory.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20752
Thanks for the clear summary, @cloud-fan. I think we want to make it easy
to support batch, and then easy to reuse those internals to support streaming
by adding new mix-in interfaces. Streaming is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20751
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/20751
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/20751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88011/
Test PASSed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20657#discussion_r172604366
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -520,4 +520,16 @@ package object config {
.checkValue(v => v
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88012/
Test FAILed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20742#discussion_r172616332
--- Diff: R/pkg/DESCRIPTION ---
@@ -57,6 +57,6 @@ Collate:
'types.R'
'utils.R'
'window.R'
-RoxygenNote: 5.0.1
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172621472
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamingDataWriterFactory.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20752
I agree that we should put a design for the streaming side on the dev list,
and I intend to do so. The streaming interfaces will remain evolving until a
design discussion about them has
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20750
**[Test build #88009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88009/testReport)**
for PR 20750 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20686
**[Test build #88023 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88023/testReport)**
for PR 20686 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20686
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/20686
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88023/
Test PASSed.
---
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/20745
Re-created the PR because something got stuck in the previous one.
cc @tdas @zsxwing @vanzin
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20750
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/20750
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/1327/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20751
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/20751
**[Test build #88012 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88012/testReport)**
for PR 20751 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20750
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172620679
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamWriter.java
---
@@ -27,6 +28,9 @@
*
* Streaming queries
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172617421
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamingDataWriterFactory.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20751
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172617442
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamWriter.java
---
@@ -27,6 +28,9 @@
*
* Streaming
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20752#discussion_r172627199
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/streaming/StreamWriter.java
---
@@ -27,6 +28,9 @@
*
* Streaming
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20687
**[Test build #88024 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88024/testReport)**
for PR 20687 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20687
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/20687
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/1331/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20752
I'm not very familiar with the streaming side, but here is my 2 cents: I
agree with @rdblue that it's unnecessary to introduce the epoch id to data
sources that don't care about streaming.
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/20687
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20752
> Right now, we're still at the point where we aren't quite sure what a
streaming API needs to look like. We're starting from basically ground zero;
the V1 streaming API just throws a DataFrame at
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20752
Sounds fair to me. I'll continue iterating on the read side, and send out a
design proposal for the write side incorporating this discussion in the next
few days.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20295
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88020/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20295
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/20750
**[Test build #88017 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88017/testReport)**
for PR 20750 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20687
**[Test build #88024 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88024/testReport)**
for PR 20687 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20748
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/20295#discussion_r172699274
--- Diff: python/pyspark/sql/udf.py ---
@@ -35,24 +37,29 @@ def _wrap_function(sc, func, returnType):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/1336/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20753
ping @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20295
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/20752
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88019/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20742
**[Test build #88021 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88021/testReport)**
for PR 20742 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20678
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88016/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20678
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/20750
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20727#discussion_r172682591
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/HadoopFileLinesReader.scala
---
@@ -42,7 +52,12 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20748
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/20748
Test FAILed.
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/1333/
Github user ssuchter commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r172707045
--- Diff: resource-managers/kubernetes/integration-tests/include/util.sh ---
@@ -0,0 +1,43 @@
+#!/usr/bin/env bash
+
+# Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/20753
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88028/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88028 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88028/testReport)**
for PR 20753 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20753#discussion_r172713356
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -133,8 +134,21 @@ case class StaticInvoke(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20685
From what I've seen in https://github.com/apache/spark/pull/15923 , this
corruption check is mostly to detect network failures, so seems like we don't
need this check for local blocks at all.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20295
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/1332/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20295
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/20697
In much of the old integration test setup, we were cloning and building
Spark into a TGZ. I wonder if we can skip both of these steps. We shouldn't
ever have to clone Spark, as we have the
Github user atongen commented on the issue:
https://github.com/apache/spark/pull/10949
I am going to close this PR because there doesn't appear to be any interest
in getting it merged. It's unfortunate, because it was a nice feature.
---
Github user atongen closed the pull request at:
https://github.com/apache/spark/pull/10949
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r172685889
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/cloud/CloudTestBackend.scala
---
Github user ssuchter commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r172691439
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/cloud/CloudTestBackend.scala
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20748
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/1334/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20748
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r172708797
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/OffHeapColumnVector.java
---
@@ -57,20 +59,20 @@
// The data stored
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20611#discussion_r172716633
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -385,8 +385,12 @@ case class LoadDataCommand(
val
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20727#discussion_r172656702
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/HadoopFileLinesReader.scala
---
@@ -42,7 +52,12 @@ class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20735#discussion_r172661657
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -736,7 +736,8 @@ private object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20752
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/20751
**[Test build #88022 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88022/testReport)**
for PR 20751 at commit
1 - 100 of 427 matches
Mail list logo