Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19514
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19513
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18747#discussion_r145411743
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -23,21 +23,72 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18747#discussion_r145410797
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -23,21 +23,72 @@ import
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19524
@holdenk, hm, should I maybe make add this to `run-tests.py` and
`run-tests-jenkins.py`? I wasn't sure where I should put this.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19488
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19521
Empty schema path probably related with this IIRC (not double checked):
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19488
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19521
LGTM too BTW.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19521
Oh, I confused with what I'm watching in these days.
For your example, Parquet also doesn't support. We may create an issue for
both Parquet/ORC on empty schema .
```scala
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19390
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/19390
**[Test build #82882 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82882/testReport)**
for PR 19390 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19390
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82882/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19390
**[Test build #82882 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82882/testReport)**
for PR 19390 at commit
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/19390
@srowen could I get a review or a merge pls?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/19390#discussion_r145397272
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -380,7 +389,8 @@
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19374
**[Test build #82881 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82881/testReport)**
for PR 19374 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19527
cc @MLnick @WeichenXu123 @jkbradley This adds a new class
`OneHotEncoderEstimator` which extends `Estimator`. Please review this when you
can. Thanks.
---
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/19374
@srowen Could I get a merge or a review pls?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/19374#discussion_r145391350
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala
---
@@ -135,22 +135,24 @@
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/19510#discussion_r145389586
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -64,6 +64,7 @@
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/19510
@windkit there is an open issue here:
https://issues.apache.org/jira/browse/SPARK-22133
Could you please add documentation of the new property along with the old
ones.
---
Github user akopich commented on the issue:
https://github.com/apache/spark/pull/18924
ping @jkbradley. Anyway, tests are passed now.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18924
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82880/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18924
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/18924
**[Test build #82880 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82880/testReport)**
for PR 18924 at commit
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/19510#discussion_r145383146
--- Diff:
resource-managers/mesos/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -152,6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19419
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/19419
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82878/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19419
**[Test build #82878 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82878/testReport)**
for PR 19419 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19419
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19419
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82876/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19419
**[Test build #82876 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82876/testReport)**
for PR 19419 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19488
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82877/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19488
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/19488
**[Test build #82877 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82877/testReport)**
for PR 19488 at commit
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r145371903
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -282,8 +348,27 @@ class LinearSVC @Since("2.2.0") (
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r145369694
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -282,8 +348,27 @@ class LinearSVC @Since("2.2.0") (
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r145371704
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -42,7 +44,26 @@ import org.apache.spark.sql.functions.{col,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19527
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/19527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82879/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19527
**[Test build #82879 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82879/testReport)**
for PR 19527 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18924
**[Test build #82880 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82880/testReport)**
for PR 18924 at commit
Github user akopich commented on the issue:
https://github.com/apache/spark/pull/18924
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19522
**[Test build #3952 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3952/testReport)**
for PR 19522 at commit
Github user akopich commented on the issue:
https://github.com/apache/spark/pull/18924
@jkbradley, no problem. The test build seems to be aborted. What's wrong?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19527
**[Test build #82879 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82879/testReport)**
for PR 19527 at commit
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/19527
[SPARK-13030][ML] Create OneHotEncoderEstimator for OneHotEncoder as
Estimator
## What changes were proposed in this pull request?
This patch adds a new class `OneHotEncoderEstimator`
Github user krishna-pandey commented on the issue:
https://github.com/apache/spark/pull/19419
@srowen @rxin Made changes to enable the X-Content-Type-Options and
X-XSS-Protection values by default. Please review.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19526
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user SimonUzL opened a pull request:
https://github.com/apache/spark/pull/19526
[SPARK-22014][SQL] removed TypeCheckFailure: slide duration <=
windowDuration
It is possible to create sampling windows in Spark Streaming, where the
duration of the window is smaller than the
Github user guoxiaolongzte commented on a diff in the pull request:
https://github.com/apache/spark/pull/19242#discussion_r145346521
--- Diff: docs/configuration.md ---
@@ -740,6 +740,20 @@ Apart from these, the following properties are also
available, and may be useful
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r145344124
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,124 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r145342355
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/HadoopUtils.scala
---
@@ -0,0 +1,119 @@
+/*
+ * Licensed to the Apache Software
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r145341631
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,124 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r145342026
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,258 @@
+/*
+ * Licensed to the Apache Software
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r145343991
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,124 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19242#discussion_r145343263
--- Diff: docs/configuration.md ---
@@ -740,6 +740,20 @@ Apart from these, the following properties are also
available, and may be useful
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19525
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82875/
Test PASSed.
---
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/19242
@srowen
Help to review the code, thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19525
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/19525
**[Test build #82875 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82875/testReport)**
for PR 19525 at commit
Github user guoxiaolongzte closed the pull request at:
https://github.com/apache/spark/pull/19360
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19360
Let's leave this closed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19523#discussion_r145340100
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -204,6 +204,7 @@ case class In(value: Expression,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19419
**[Test build #82878 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82878/testReport)**
for PR 19419 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19488
**[Test build #82877 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82877/testReport)**
for PR 19488 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19488
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19459#discussion_r145336060
--- Diff: python/pyspark/sql/session.py ---
@@ -414,6 +415,39 @@ def _createFromLocal(self, data, schema):
data = [schema.toInternal(row)
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/19525#discussion_r145331849
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/linalg/JsonMatrixConverter.scala ---
@@ -0,0 +1,79 @@
+/*
+ * Licensed to the Apache Software
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/19525#discussion_r145333064
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -122,17 +124,33 @@ private[ml] object Param {
/** Decodes a param
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19419
**[Test build #82876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82876/testReport)**
for PR 19419 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19459#discussion_r145334576
--- Diff: python/pyspark/sql/session.py ---
@@ -414,6 +415,39 @@ def _createFromLocal(self, data, schema):
data = [schema.toInternal(row) for
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19360
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19469
@ssaavedra , yes I think so. with the pull-in of k8s support, I would guess
more configurations need to be added to exclusion rule. With current solution,
one by one PR doesn't make so sense. We
GitHub user guoxiaolongzte reopened a pull request:
https://github.com/apache/spark/pull/19360
[SPARK-22139][CORE]Remove the variable which is never used in
SparkConf.scala
## What changes were proposed in this pull request?
Remove the variable which is never used in
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19522
**[Test build #3952 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3952/testReport)**
for PR 19522 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19509
LGTM, just one minor comment.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19522
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82873/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19488
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82874/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19488
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/19522
**[Test build #82873 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82873/testReport)**
for PR 19522 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19522
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/19488
**[Test build #82874 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82874/testReport)**
for PR 19488 at commit
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/19525#discussion_r145330685
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -122,17 +124,33 @@ private[ml] object Param {
/** Decodes a param
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19525
**[Test build #82875 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82875/testReport)**
for PR 19525 at commit
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/19525
[SPARK-22289] [ML] Add JSON support for Matrix parameters (LR with
coefficients bound)
## What changes were proposed in this pull request?
jira:
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/19509#discussion_r145329972
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -347,6 +347,10 @@ package object config {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19508#discussion_r145329814
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/compression/compressionSchemes.scala
---
@@ -495,6 +474,8 @@ private[columnar]
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18664
Will take another look tomorrow. Thanks for your work! @BryanCutler
---
-
To unsubscribe, e-mail:
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/18664
I'd cc @gatorsmile, @cloud-fan for final check.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18664
LGTM too
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19524
Looks reasonable, my only thing is putting in a forced exited inside of
shellutil.py is sort of a random place to the version check.
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19509
I see, thanks for the explanation. I didn't think about such scenario.
---
-
To unsubscribe, e-mail:
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/19507
@ajbozarth
Sorry, upload the code before I accidentally withdrew the parenthesis. I
rejoined the parenthesis. I have fixed it.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19524
seems fine.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18664#discussion_r145327316
--- Diff: python/pyspark/serializers.py ---
@@ -259,11 +261,13 @@ def load_stream(self, stream):
"""
Deserialize ArrowRecordBatches
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19508#discussion_r145323960
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/compression/compressionSchemes.scala
---
@@ -495,6 +474,8 @@ private[columnar]
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19508#discussion_r145323573
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/ColumnAccessor.scala
---
@@ -63,9 +63,6 @@ private[columnar] abstract class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19488
**[Test build #82874 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82874/testReport)**
for PR 19488 at commit
201 - 300 of 300 matches
Mail list logo