Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20742
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20844#discussion_r175230004
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -396,9 +396,11 @@ case class RangeExec(range:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20742
**[Test build #88324 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88324/testReport)**
for PR 20742 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20803
Sorry I didn't clarify it clearly enough. I was not suggesting to show sql
text for all of these cases, but tried to raise a discussion about when we
should show sql text. e.g. for `df.count()`
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20579
After some more thoughts, we should try our best to not introduce behavior
change to existing data sources. How about we only add this check for
file-based data sources(all of them are built-in)?
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20726
LGTM, pending jenkins
---
-
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/20726
**[Test build #88329 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88329/testReport)**
for PR 20726 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20726
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/20726
**[Test build #88328 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88328/testReport)**
for PR 20726 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20726
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88328/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20726
**[Test build #88328 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88328/testReport)**
for PR 20726 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20745
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/20745
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88323/
Test PASSed.
---
Github user tedyu commented on the issue:
https://github.com/apache/spark/pull/20767
I did a quick search for 'apache commons pool metrics' which didn't show up
directly related links.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20745
**[Test build #88323 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88323/testReport)**
for PR 20745 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20726#discussion_r175222350
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/SupportsReportPartitioning.java
---
@@ -23,6 +23,11 @@
/**
* A mix in
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/20767
@tedyu @zsxwing My thoughts on this is that we should consider migrating to
something like Apache Common Pool (assuming it does not require additional
maven libraries), which might be less maintenance
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20752
There've been no comments on the doc. Should we move forward with this PR?
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20796#discussion_r175215287
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1279,4 +1280,57 @@ class CSVSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20343
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88321/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20343
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/20343
**[Test build #88321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88321/testReport)**
for PR 20343 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20579
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/20579
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/1572/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20579
**[Test build #88327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88327/testReport)**
for PR 20579 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20809
Can you provide more information in the bug report? e.g. a sample
application and a sample error.
I don't think this is the correct change, but without your use case I'm not
sure what the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20327
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/20327
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88318/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20327
**[Test build #88318 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88318/testReport)**
for PR 20327 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20796
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20796
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88320/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20796
**[Test build #88320 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88320/testReport)**
for PR 20796 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88317/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88317 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88317/testReport)**
for PR 20659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20807
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/20807
**[Test build #88326 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88326/testReport)**
for PR 20807 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20807
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88326/
Test PASSed.
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20742
lgtm
---
-
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/20807
**[Test build #88326 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88326/testReport)**
for PR 20807 at commit
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/20807
@vanzin sorry, one useless comment left in the code. Just removed.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20726
**[Test build #88325 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88325/testReport)**
for PR 20726 at commit
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20726
Addressed last comment. Sorry it took me so long.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20742
**[Test build #88324 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88324/testReport)**
for PR 20742 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20742
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1571/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20742
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20745
**[Test build #88323 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88323/testReport)**
for PR 20745 at commit
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20745#discussion_r175186820
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSinkSuite.scala
---
@@ -405,4 +406,52 @@ class FileStreamSinkSuite
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/17774
merged to master
Thanks @arzt !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20745#discussion_r175186800
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSinkSuite.scala
---
@@ -405,4 +406,52 @@ class FileStreamSinkSuite
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/20777#discussion_r175184951
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/CountVectorizer.scala ---
@@ -70,19 +70,21 @@ private[feature] trait CountVectorizerParams
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/20777#discussion_r175184795
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/CountVectorizer.scala ---
@@ -70,19 +70,21 @@ private[feature] trait CountVectorizerParams
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/20777#discussion_r175184503
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/CountVectorizer.scala ---
@@ -70,19 +70,21 @@ private[feature] trait CountVectorizerParams
Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r175184304
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -669,11 +672,42 @@ object
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16770
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16770
Looks good to me, merged to master. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
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/20847
**[Test build #88322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88322/testReport)**
for PR 20847 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
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/1570/
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20847
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user DylanGuedes commented on a diff in the pull request:
https://github.com/apache/spark/pull/20788#discussion_r175181523
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -437,10 +437,12 @@ def hint(self, name, *parameters):
if not isinstance(name, str):
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16770#discussion_r175177201
--- Diff: python/pyspark/ml/feature.py ---
@@ -497,66 +544,61 @@ def setMinTF(self, value):
"""
return self._set(minTF=value)
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16770#discussion_r175179623
--- Diff: python/pyspark/ml/feature.py ---
@@ -565,6 +607,20 @@ def vocabulary(self):
"""
return self._call_java("vocabulary")
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16770#discussion_r175180172
--- Diff: python/pyspark/ml/tests.py ---
@@ -1980,8 +1997,8 @@ def test_java_params(self):
pyspark.ml.regression]
for
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16770#discussion_r175179845
--- Diff: python/pyspark/ml/tests.py ---
@@ -640,6 +640,34 @@ def test_count_vectorizer_with_binary(self):
feature, expected = r
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/20745#discussion_r175178609
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSinkSuite.scala
---
@@ -405,4 +406,52 @@ class FileStreamSinkSuite extends
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/20745#discussion_r175178230
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSinkSuite.scala
---
@@ -405,4 +406,52 @@ class FileStreamSinkSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20796
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20796
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88316/
Test FAILed.
---
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20767
@tedyu that's a good idea. Could you create a ticket? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20705
Thank you, @gatorsmile , @HyukjinKwon , @bersprockets .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20796
**[Test build #88316 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88316/testReport)**
for PR 20796 at commit
Github user tedyu commented on the issue:
https://github.com/apache/spark/pull/20767
@tdas
Do you think a follow on JIRA can be logged for adding metrics for the
cache operations ?
Thanks
---
-
To
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20767
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20742#discussion_r175171592
--- Diff: docs/security.md ---
@@ -3,47 +3,291 @@ layout: global
displayTitle: Spark Security
title: Security
---
+* This will become a
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20742#discussion_r175173523
--- Diff: docs/security.md ---
@@ -182,54 +582,70 @@ configure those ports.
-### HTTP Security Headers
-Apache Spark can be
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20742#discussion_r175170426
--- Diff: docs/security.md ---
@@ -3,47 +3,291 @@ layout: global
displayTitle: Spark Security
title: Security
---
+* This will become a
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20767
@tdas this is much simpler!!! LGTM. Merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88315/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20847
**[Test build #88315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88315/testReport)**
for PR 20847 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20343
**[Test build #88321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88321/testReport)**
for PR 20343 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20822
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/20822
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user squito commented on the issue:
https://github.com/apache/spark/pull/19041
Thanks @brad-kaiser -- want to re-iterate my comment from Feb 2nd, I think
that is really the most important part to address before getting into the
details of the current implementation:
>
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/19431
@akonopko thanks for this, if you can resolve merge conflict I think we can
get this in
---
-
To unsubscribe, e-mail:
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r175164254
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -246,6 +251,38 @@ class BlockManagerMasterEndpoint(
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20841#discussion_r175163502
--- Diff: python/pyspark/sql/tests.py ---
@@ -2504,6 +2504,10 @@ def test_conf(self):
spark.conf.unset("bogo")
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20807
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/20807
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88319/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20807
**[Test build #88319 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88319/testReport)**
for PR 20807 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20750
---
-
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/17774
---
-
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/20796
**[Test build #88320 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88320/testReport)**
for PR 20796 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20750
@cloud-fan has some issue with his mac, so I will be merging :)...
Thanks for the reviews!
---
-
To unsubscribe,
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/20796
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20750
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user attilapiros commented on the issue:
https://github.com/apache/spark/pull/20046
gentle ping @gatorsmile @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20788#discussion_r175156065
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -437,10 +437,12 @@ def hint(self, name, *parameters):
if not isinstance(name, str):
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20795#discussion_r175159061
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1192,11 +1195,24 @@ class Analyzer(
* @see
101 - 200 of 412 matches
Mail list logo