Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/20091
Thanks for coding it up @jiangxb1987 !
So if I understand it correctly, the requirements where the PR helps with
are :
* Max partitioner is not eligible since it is atleast an order smal
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20295
How do we turn a single group column to a series? just repeat the group
column?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20177
can you fix the test?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20026
cc @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19862
I don't agree this is a small change, and users using spark prior to 2.0
won't get this patch, as we don't backport performance improvement patches.
Overall this patch won't bring much ben
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/20091#discussion_r162552121
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -67,31 +69,32 @@ object Partitioner {
None
}
-if (i
Github user fjh100456 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20087#discussion_r162551602
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/CompressionCodecSuite.scala
---
@@ -0,0 +1,321 @@
+/*
+ * Licensed to the Apache Soft
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19583
also cc @JoshRosen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20087#discussion_r162551462
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/CompressionCodecSuite.scala
---
@@ -0,0 +1,321 @@
+/*
+ * Licensed to the Apache Sof
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20087#discussion_r162551351
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/CompressionCodecSuite.scala
---
@@ -0,0 +1,321 @@
+/*
+ * Licensed to the Apache Sof
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19054#discussion_r162551159
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/EnsureRequirements.scala
---
@@ -220,45 +220,76 @@ case class EnsureRequirement
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19054#discussion_r162550613
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/EnsureRequirements.scala
---
@@ -220,45 +220,76 @@ case class EnsureRequirement
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19175
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/19175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86374/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19175
**[Test build #86374 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86374/testReport)**
for PR 19175 at commit
[`d1133ca`](https://github.com/apache/spark/commit/d
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20297#discussion_r162550217
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/ChildProcAppHandle.java ---
@@ -48,14 +48,16 @@ public synchronized void disconnect() {
Github user Ngone51 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19285#discussion_r162549759
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -261,37 +263,93 @@ private[spark] class MemoryStore(
/
Github user Ngone51 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19285#discussion_r162548350
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -162,26 +162,33 @@ private[spark] class MemoryStore(
}
Github user Ngone51 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19285#discussion_r162548052
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -233,17 +235,13 @@ private[spark] class MemoryStore(
}
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
It's just a refactor so I'd like to target it for 2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20091#discussion_r162549412
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -67,31 +69,32 @@ object Partitioner {
None
}
-if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20316
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/28/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20316
**[Test build #86377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86377/testReport)**
for PR 20316 at commit
[`b3fb8f2`](https://github.com/apache/spark/commit/b3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20316
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20316
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/20091#discussion_r162548620
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -67,31 +69,32 @@ object Partitioner {
None
}
-if (i
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20277
**[Test build #86376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86376/testReport)**
for PR 20277 at commit
[`3972093`](https://github.com/apache/spark/commit/39
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 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/testing-k8s-prb-make-spark-distribution/27/
Test
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19340
@viirya yes you're right in your analysis. Where in the doc should we put
this?
@srowen please if you.think this.is.ok, may you start a build? Thanks.
---
---
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20297#discussion_r162548031
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherConnection.java ---
@@ -95,15 +95,15 @@ protected synchronized void send(Message ms
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20324#discussion_r162547532
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameStatSuite.scala ---
@@ -154,24 +154,24 @@ class DataFrameStatSuite extends QueryTest with
Sh
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162547306
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ColumnarBatchScan.scala
---
@@ -50,7 +50,14 @@ private[sql] trait ColumnarBatchScan exte
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18277
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/18277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86375/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #86375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86375/testReport)**
for PR 18277 at commit
[`8c88595`](https://github.com/apache/spark/commit/8
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/86372/
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/20277
**[Test build #86372 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86372/testReport)**
for PR 20277 at commit
[`37c82e6`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20327
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20327
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user gerashegalov opened a pull request:
https://github.com/apache/spark/pull/20327
[SPARK-12963][CORE] NM host for driver end points
## What changes were proposed in this pull request?
Driver end points on YARN in the cluster mode are potentially bound to
incorrect
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18277
This change looks reasonable to me for now. But I'm also concerned about
the behavior change. A note into release notes should be good or maybe we need
a note at migration guide in `RDD Programming G
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #86375 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86375/testReport)**
for PR 18277 at commit
[`8c88595`](https://github.com/apache/spark/commit/8c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20316
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/20316
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86367/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20316
**[Test build #86367 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86367/testReport)**
for PR 20316 at commit
[`e05fb06`](https://github.com/apache/spark/commit/e
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18277
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20326
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20326
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user gerashegalov opened a pull request:
https://github.com/apache/spark/pull/20326
[SPARK-23155][DEPLOY] log.server.url links in SHS
## What changes were proposed in this pull request?
Ensure driver/executor log availability via Spark History Server UI even if
the o
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20323
> If Travis CI can not handle the full traffic of Apache Spark PRs, we may
run it for only Java code change PRs.
@dongjoon-hyun, do you know if Travis CI supports exclusion/inclusion of
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20325
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20325
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19892
Iâm generally ok with these small python api wrapper additions getting
merged as long as the risk of breaking anything is low - and here it is
since itâs just api parity
On Fri, 19 Jan
GitHub user brandonJY opened a pull request:
https://github.com/apache/spark/pull/20325
[SPARK-22808][DOCS] add insertInto when save hive built dataframe
## What changes were proposed in this pull request?
based on https://issues.apache.org/jira/browse/SPARK-22808 &
http
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20324
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/20324
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86368/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20324
**[Test build #86368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86368/testReport)**
for PR 20324 at commit
[`4eef6d6`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20298
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/20298
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86365/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20298
**[Test build #86365 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86365/testReport)**
for PR 20298 at commit
[`38916f7`](https://github.com/apache/spark/commit/3
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19285
Are we targeting this to 2.3 or 2.4?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162536698
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ColumnarBatchScan.scala
---
@@ -50,7 +50,14 @@ private[sql] trait ColumnarBatchScan extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162536737
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/VectorizedHashMapGenerator.scala
---
@@ -127,8 +127,14 @@ class VectorizedHashMap
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20316
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86369/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20316
Merged build finished. Test FAILed.
---
-
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/20316
**[Test build #86369 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86369/testReport)**
for PR 20316 at commit
[`b3fb8f2`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20275
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86373/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20275
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/20275
**[Test build #86373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86373/testReport)**
for PR 20275 at commit
[`f3a4329`](https://github.com/apache/spark/commit/f
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162534791
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
---
@@ -53,166 +41,83 @@ public int numNulls() {
@Override
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19301
I believe this has been fixed, can we close it?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19293
if it's too hard to write a UT, can we have a code snippet to reproduce
this bug and put it in PR description?
---
-
To unsubs
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19285
overall looks good
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19285#discussion_r162534339
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -261,37 +263,93 @@ private[spark] class MemoryStore(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19285#discussion_r162534289
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -162,26 +162,33 @@ private[spark] class MemoryStore(
}
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19892
I mean I think it might have a chance, generally speaking we've allowed
outstanding PRs to be merged after the freeze. Since there are outstanding
blockers on the branch preventing us from cutting R
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19175
**[Test build #86374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86374/testReport)**
for PR 19175 at commit
[`d1133ca`](https://github.com/apache/spark/commit/d1
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18277
Jenkins OK to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19175
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19420
I love more examples, but is there a place we plan to put this in the
documentation?
---
-
To unsubscribe, e-mail: reviews-unsub
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19420#discussion_r162532658
--- Diff:
examples/src/main/java/org/apache/spark/examples/sql/hive/JavaSparkHiveExample.java
---
@@ -124,6 +124,13 @@ public static void main(String[] arg
Github user xubo245 commented on the issue:
https://github.com/apache/spark/pull/20260
I will fix the error of this PR after
https://github.com/apache/spark/pull/20249#issuecomment-358720962 merged
---
-
To unsubscr
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19420
Jenkins OK to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17185
I agree it's a valid use case, do you wanna bring it up to date? sorry for
the delay!
---
-
To unsubscribe, e-mail: reviews-un
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17123
cc @WeichenXu123
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19872#discussion_r162532163
--- Diff: python/pyspark/sql/tests.py ---
@@ -4279,6 +4273,425 @@ def test_unsupported_types(self):
df.groupby('id').apply(f).collect(
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19876
also maybe @dbtsai ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20306
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162531810
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ColumnVector.java ---
@@ -152,19 +198,11 @@ public final ColumnarRow getStruct(int rowI
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20306
thanks, merging to master/2.3!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162531441
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
---
@@ -53,166 +41,83 @@ public int numNulls() {
@Overri
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162531459
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
---
@@ -53,166 +41,83 @@ public int numNulls() {
@Overri
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20091#discussion_r162531289
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -67,31 +69,32 @@ object Partitioner {
None
}
-if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20025
**[Test build #86370 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86370/testReport)**
for PR 20025 at commit
[`5b3c06f`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86370/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20025
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18983
LGTM, although I'm not very familiar with the thrift server code...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apa
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/testing-k8s-prb-make-spark-distribution/26/
Test
1 - 100 of 866 matches
Mail list logo