New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-31550][SQL][DOCS] Set nondeterministic configurations with general meanings in sql configuration doc #28322
Conversation
…eral meanings in sql configuration doc
Test build #121721 has finished for PR 28322 at commit
|
Jenkins, retest this please |
Test build #121729 has finished for PR 28322 at commit
|
retest this please |
Test build #121750 has finished for PR 28322 at commit
|
cc: @dongjoon-hyun @cloud-fan @maropu too, thanks. |
sql/core/src/main/scala/org/apache/spark/sql/api/python/PythonSQLUtils.scala
Outdated
Show resolved
Hide resolved
sql/core/src/main/scala/org/apache/spark/sql/api/python/PythonSQLUtils.scala
Outdated
Show resolved
Hide resolved
Test build #121800 has finished for PR 28322 at commit
|
sql/core/src/main/scala/org/apache/spark/sql/api/python/PythonSQLUtils.scala
Outdated
Show resolved
Hide resolved
Test build #121830 has finished for PR 28322 at commit
|
Test build #121843 has finished for PR 28322 at commit
|
Merged to master and branch-3.0. |
…eral meanings in sql configuration doc ### What changes were proposed in this pull request? ```scala spark.sql.session.timeZone spark.sql.warehouse.dir ``` these 2 configs are nondeterministic and vary with environments Besides, reflect code in `gen-sql-config-docs.py` via #28274 (comment) and `configuration.md` via #28274 (comment) ### Why are the changes needed? doc fix ### Does this PR introduce any user-facing change? no ### How was this patch tested? verify locally ![image](https://user-images.githubusercontent.com/8326978/80179099-5e7da200-8632-11ea-803f-d47a93151869.png) Closes #28322 from yaooqinn/SPARK-31550. Authored-by: Kent Yao <yaooqinn@hotmail.com> Signed-off-by: HyukjinKwon <gurwls223@apache.org> (cherry picked from commit 5ba467c) Signed-off-by: HyukjinKwon <gurwls223@apache.org>
late LGTM, thanks for the update, @yaooqinn |
What changes were proposed in this pull request?
these 2 configs are nondeterministic and vary with environments
Besides, reflect code in
gen-sql-config-docs.py
via #28274 (comment) andconfiguration.md
via #28274 (comment)Why are the changes needed?
doc fix
Does this PR introduce any user-facing change?
no
How was this patch tested?
verify locally