Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20297#discussion_r162693731
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/ChildProcAppHandle.java ---
@@ -48,14 +48,16 @@ public synchronized void disconnect() {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20297#discussion_r162693890
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherConnection.java ---
@@ -95,15 +95,15 @@ protected synchronized void send(Message msg) thro
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20297#discussion_r162694174
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherServer.java ---
@@ -331,23 +358,27 @@ protected void handle(Message msg) throws IOExceptio
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20297#discussion_r162694343
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherServer.java ---
@@ -331,23 +358,27 @@ protected void handle(Message msg) throws IOExceptio
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20297
**[Test build #4066 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4066/testReport)**
for PR 20297 at commit
[`95bac27`](https://github.com/apache/spark/commit/9
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20297
I kicked an extra couple of builds aside from the one that should
auto-trigger.
---
-
To unsubscribe, e-mail: reviews-unsubscr...
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20297
**[Test build #4067 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4067/testReport)**
for PR 20297 at commit
[`95bac27`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20297
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20297
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/45/
Test
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20314
Merging to master / 2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20297
**[Test build #86395 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86395/testReport)**
for PR 20297 at commit
[`95bac27`](https://github.com/apache/spark/commit/95
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20314
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20185
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20328
An late LGTM! :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17894
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/46/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17894
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user liufengdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/20025#discussion_r162698093
--- Diff:
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/SessionManager.java
---
@@ -80,7 +76,6 @@ public synchronized void init
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20091
@mridulm Great write up! Yeah it's exactly that you described, and I've
copied them to the PR description.
---
-
To unsubscr
Github user liufengdb commented on the issue:
https://github.com/apache/spark/pull/18983
LGTM! It is only created once though.
Frankly, we should completely remove the implementation of `newSession()`
method.
---
---
GitHub user sethah opened a pull request:
https://github.com/apache/spark/pull/20332
[SPARK-23138][ML][DOC] Multiclass summary example and user guide
## What changes were proposed in this pull request?
User guide and examples are updated to reflect multiclass logistic
regre
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/20332
@jkbradley @MLnick
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20332
**[Test build #86396 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86396/testReport)**
for PR 20332 at commit
[`cb6c811`](https://github.com/apache/spark/commit/cb
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17123#discussion_r162703633
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -105,20 +106,21 @@ final class Bucketizer @Since("1.4.0")
(@Since("
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17123#discussion_r162703711
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -171,23 +176,23 @@ object Bucketizer extends
DefaultParamsReadable[
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86396/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20332
**[Test build #86396 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86396/testReport)**
for PR 20332 at commit
[`cb6c811`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
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/47/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19993
Since RC1 for 2.3 failed, it'd be great to get this into 2.3. @mgaido91 do
you mind if I send my comments along with a PR to update this PR of yours? I'm
rushing because of the time pressure to
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/17123
But, pls resolve conflicts first. :) Bucketizer add multiple column support
so the code is different now.
---
-
To unsubscr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20331
**[Test build #86392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86392/testReport)**
for PR 20331 at commit
[`f7693f0`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20331
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20331
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86392/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
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/48/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20332
**[Test build #86397 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86397/testReport)**
for PR 20332 at commit
[`58d973e`](https://github.com/apache/spark/commit/58
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20331
**[Test build #86393 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86393/testReport)**
for PR 20331 at commit
[`f7693f0`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20331
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86393/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20331
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20138
as RC1 failed and RC2 is going to be cut soon, I'm going to merge this to
master & 2.3
---
-
To unsubscribe, e-mail: reviews-unsu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20138
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20284
even though we don't *know* of this causing a bug in 2.3, I still think we
should merge it in there just because there may be some case we aren't thinking
of, and this is a relatively small, safe fix
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20203
**[Test build #86398 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86398/testReport)**
for PR 20203 at commit
[`41dd7bb`](https://github.com/apache/spark/commit/41
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20330#discussion_r162712767
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -427,23 +435,21 @@ private[ui] class JobDataSource(
val formattedD
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20284
---
-
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/20332
**[Test build #86397 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86397/testReport)**
for PR 20332 at commit
[`58d973e`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20332
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86397/
Test PASSed.
---
Github user attilapiros commented on the issue:
https://github.com/apache/spark/pull/20203
The node blacklisting is tested by unit tests:
- HistoryServerSuite
- TaskSetBlacklistSuite
- AppStatusListenerSuite
And manually with a 2 node cluster:
https://issues.apache
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r162714257
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetBlacklistSuite.scala ---
@@ -59,31 +60,55 @@ class TaskSetBlacklistSuite extends SparkFunSuit
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r162715788
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -249,6 +249,16 @@ object ParamValidators {
def arrayLengthGt[T](lowerBo
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r162716271
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetBlacklistSuite.scala ---
@@ -59,31 +60,55 @@ class TaskSetBlacklistSuite extends SparkFu
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20203
btw another way you could test out having a bad host would be something
like this (untested):
```scala
import org.apache.spark.SparkEnv
val hosts = sc.parallelize(1 to 1, 100
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19993#discussion_r162717142
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -201,9 +184,13 @@ final class Bucketizer @Since("1.4.0")
(@Since("1.4.
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19993#discussion_r162717263
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -166,6 +167,8 @@ private[ml] object Param {
@DeveloperApi
object Par
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19993
@jkbradley sure no problem, let me know how I can help.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19993#discussion_r162719519
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -201,9 +184,13 @@ final class Bucketizer @Since("1.4.0")
(@Since("1.4.0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20330
**[Test build #86399 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86399/testReport)**
for PR 20330 at commit
[`d5fdabb`](https://github.com/apache/spark/commit/d5
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20330#discussion_r162721177
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -427,23 +435,21 @@ private[ui] class JobDataSource(
val formattedDura
Github user smurakozi commented on the issue:
https://github.com/apache/spark/pull/20235
@jkbradley could you check out this change, please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user smurakozi commented on the issue:
https://github.com/apache/spark/pull/20319
@jkbradley could you check out this change, please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/20091
@jiangxb1987 Thanks for clarifying, looks good to me - I will merge it
later today evening (assuming someone else does not before :) )
---
-
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20091
@mridulm Thank you!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20203
**[Test build #86400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86400/testReport)**
for PR 20203 at commit
[`cf6e0c9`](https://github.com/apache/spark/commit/cf
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20201#discussion_r162732939
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/DataSourceV2Suite.scala
---
@@ -95,6 +96,34 @@ class DataSourceV2Suite extends QueryTe
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20201#discussion_r162733141
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/ClusteredDistribution.java
---
@@ -0,0 +1,34 @@
+/*
+ * Licensed to the
Github user sameeragarwal commented on the issue:
https://github.com/apache/spark/pull/20299
LGTM. Merging this to master/2.3. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20201#discussion_r162733351
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourcePartitioning.scala
---
@@ -0,0 +1,49 @@
+/*
+ * Licens
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20201#discussion_r162733463
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/Distribution.java
---
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the Apache So
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20201#discussion_r162733629
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/Partitioning.java
---
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache So
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20201#discussion_r162733684
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/Partitioning.java
---
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache So
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20299
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/20333
[SPARK-23087][SQL] CheckCartesianProduct too restrictive when condition is
false/null
## What changes were proposed in this pull request?
CheckCartesianProduct raises an AnalysisException
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20333
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/49/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20333
**[Test build #86401 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86401/testReport)**
for PR 20333 at commit
[`9c88781`](https://github.com/apache/spark/commit/9c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20333
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20277
**[Test build #86394 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86394/testReport)**
for PR 20277 at commit
[`3972093`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86394/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20277
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20297
**[Test build #86395 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86395/testReport)**
for PR 20297 at commit
[`95bac27`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20297
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20297
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86395/
Test PASSed.
---
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20177#discussion_r162746808
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -31,9 +31,9 @@ case class AnalyzeTableCommand
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20297
**[Test build #4066 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4066/testReport)**
for PR 20297 at commit
[`95bac27`](https://github.com/apache/spark/commit/
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19993#discussion_r162747183
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -201,9 +184,13 @@ final class Bucketizer @Since("1.4.0")
(@Since("1.4.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20297
**[Test build #4067 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4067/testReport)**
for PR 20297 at commit
[`95bac27`](https://github.com/apache/spark/commit/
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162747998
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
---
@@ -33,18 +33,6 @@
private final ArrowVectorAccessor acces
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162748352
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
---
@@ -55,164 +43,82 @@ public void close() {
if (childColu
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20177#discussion_r162749089
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala ---
@@ -154,11 +155,17 @@ abstract class SQLViewSuite extends QueryT
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20331
**[Test build #86402 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86402/testReport)**
for PR 20331 at commit
[`b83f859`](https://github.com/apache/spark/commit/b8
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20177
We shall also update for `AnalyzePartitionCommand` and
`AnalyzeColumnCommand`.
---
-
To unsubscribe, e-mail: reviews-unsubsc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20331
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/50/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20203
**[Test build #86398 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86398/testReport)**
for PR 20203 at commit
[`41dd7bb`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20331
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20203
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20203
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86398/
Test FAILed.
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20203
@attilapiros test failures look real (you probably just need to regenerate
some of those expectations).
---
-
To unsubscribe, e-m
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20316#discussion_r162753323
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ColumnarBatch.java ---
@@ -96,16 +90,6 @@ public void setNumRows(int numRows) {
*/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20331
**[Test build #86402 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86402/testReport)**
for PR 20331 at commit
[`b83f859`](https://github.com/apache/spark/commit/b
201 - 300 of 372 matches
Mail list logo