Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20657#discussion_r173383032
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -520,4 +520,16 @@ package object config {
.checkValue(v =>
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20778#discussion_r173382375
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -392,8 +392,44 @@ case class NewInstance(
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20657#discussion_r173380826
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/security/AMCredentialRenewer.scala
---
@@ -18,221 +18,160 @@ package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
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/20757
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88111/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20757
**[Test build #88111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88111/testReport)**
for PR 20757 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20779
Under the current situation, I think that we have to create very very huge
query since we made codegen stable.
---
-
To
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20779
I'd rather avoid to change it, because modifying it we can always trigger
exceptions like the constant pool size limit. Can't we repro this without
changing that value?
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20759#discussion_r173379554
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -19,6 +19,7 @@ by a small set of latent factors that can be used to
predict missing entries.
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20779
Good point. If we could set a specific value to
`GENERATED_CLASS_SIZE_THRESHOLD`, we can add a test case.
How do we make `GENERATED_CLASS_SIZE_THRESHOLD` non-final for testing? For
example, by
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20779
Use the example in the PR description?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20779
@kiszk can we add a test case for this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20754
**[Test build #88117 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88117/testReport)**
for PR 20754 at commit
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/20781
cc @vanzin @tgravescs @cloud-fan @djvulee
Could you please help review this ?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20779
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/20779
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88110/
Test PASSed.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20754
Jenkins, ok to test
---
-
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/20779
**[Test build #88110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88110/testReport)**
for PR 20779 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19222
ping @cloud-fan
---
-
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/20777
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88115/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20777
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/20777
**[Test build #88115 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88115/testReport)**
for PR 20777 at commit
Github user goungoun closed the pull request at:
https://github.com/apache/spark/pull/20782
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20780
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/20780
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88112/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20780
**[Test build #88112 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88112/testReport)**
for PR 20780 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20781
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88116/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20781
**[Test build #88116 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88116/testReport)**
for PR 20781 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20781
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/20782
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/20782
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/20747
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user goungoun opened a pull request:
https://github.com/apache/spark/pull/20782
[SPARK-SPARK-23627][SQL] Provide isEmpty in DataSet
## What changes were proposed in this pull request?
This PR adds a isEmpty in DataSet
## How was this patch tested?
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20779
Looks good.
---
-
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/20781
**[Test build #88116 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88116/testReport)**
for PR 20781 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20781
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/1422/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20781
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user jinxing64 opened a pull request:
https://github.com/apache/spark/pull/20781
[SPARK-23637][YARN]Yarn might allocate more resource if a same executor is
killed multiple times.
## What changes were proposed in this pull request?
`YarnAllocator` uses
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/20776
LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20778#discussion_r173370038
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -392,8 +392,44 @@ case class NewInstance(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20777
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/1421/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20777
**[Test build #88115 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88115/testReport)**
for PR 20777 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20777
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20777#discussion_r173369004
--- Diff: python/pyspark/ml/feature.py ---
@@ -465,26 +522,26 @@ class CountVectorizer(JavaEstimator, HasInputCol,
HasOutputCol, JavaMLReadable,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20771
**[Test build #88114 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88114/testReport)**
for PR 20771 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20771
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/20771
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/1420/
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20780#discussion_r173366467
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/sources/HadoopFsRelationTest.scala
---
@@ -335,7 +335,7 @@ abstract class
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20779
cc: @hvanhovell @mgaido91
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20771#discussion_r173364839
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -501,12 +502,22 @@ case class LambdaVariable(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20756
**[Test build #88113 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88113/testReport)**
for PR 20756 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20756
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/1419/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20756
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20756#discussion_r173363964
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -1261,8 +1261,39 @@ case class
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20753#discussion_r173361869
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ObjectExpressionsSuite.scala
---
@@ -95,6 +162,21 @@ class
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20753#discussion_r173361648
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -188,8 +189,30 @@ case class StaticInvoke(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
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/20780
**[Test build #88112 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88112/testReport)**
for PR 20780 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20780
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/1417/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
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/1418/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20780
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/20780
[MINOR] [SQL] [TEST] Create table using `dataSourceName` in
`HadoopFsRelationTest`
## What changes were proposed in this pull request?
This PR fixes a minor issue in
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20778#discussion_r173361185
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ObjectExpressionsSuite.scala
---
@@ -138,4 +154,40 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20757
**[Test build #88111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88111/testReport)**
for PR 20757 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20757#discussion_r173361092
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -121,6 +121,19 @@ object ScalaReflection extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
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/1416/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20779
**[Test build #88110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88110/testReport)**
for PR 20779 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20779
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/20779
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/1415/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20778
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88108/
Test PASSed.
---
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/20779
[SPARK-23598][SQL] Make methods in BufferedRowIterator public to avoid
compilation for a large query
## What changes were proposed in this pull request?
This PR fixes compilation error
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20778
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/20778
**[Test build #88108 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88108/testReport)**
for PR 20778 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20705#discussion_r173358166
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/MetastoreDataSourcesSuite.scala
---
@@ -852,52 +846,52 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20767
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/20767
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88109/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20767
**[Test build #88109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88109/testReport)**
for PR 20767 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
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/20757
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88107/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20757
**[Test build #88107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88107/testReport)**
for PR 20757 at commit
Github user gengliangwang closed the pull request at:
https://github.com/apache/spark/pull/20766
---
-
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/20767
**[Test build #88109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88109/testReport)**
for PR 20767 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20767
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/1414/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20767
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20767#discussion_r173351789
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaDataConsumer.scala
---
@@ -342,80 +401,65 @@ private[kafka010] object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20744
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/20744
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88105/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20744
**[Test build #88105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88105/testReport)**
for PR 20744 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20756#discussion_r173346317
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -1261,8 +1261,39 @@ case class
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/20770
@squito is the master of DAGSchedulerSuite, and can provide you the best
advice on changing or adding to the existing DAGSchedulerSuite. I'll be back
from skiing next week and try to find some
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20776
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/20776
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88104/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20776
**[Test build #88104 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88104/testReport)**
for PR 20776 at commit
Github user bersprockets commented on a diff in the pull request:
https://github.com/apache/spark/pull/20705#discussion_r173331220
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/MetastoreDataSourcesSuite.scala
---
@@ -591,7 +591,7 @@ class MetastoreDataSourcesSuite
Github user bersprockets commented on a diff in the pull request:
https://github.com/apache/spark/pull/20705#discussion_r173332327
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/MetastoreDataSourcesSuite.scala
---
@@ -852,52 +846,52 @@ class MetastoreDataSourcesSuite
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20774
cc: @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20767#discussion_r173341089
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaDataConsumer.scala
---
@@ -342,80 +401,65 @@ private[kafka010] object
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20767#discussion_r173340037
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaDataConsumer.scala
---
@@ -342,80 +401,65 @@ private[kafka010] object
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20753#discussion_r173339081
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ObjectExpressionsSuite.scala
---
@@ -95,6 +162,21 @@ class
1 - 100 of 407 matches
Mail list logo