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
[MINOR][SQL] Avoid hardcoded configuration keys in SQLConf's doc
#22877
Conversation
Test build #98209 has finished for PR 22877 at commit
|
@@ -408,15 +408,16 @@ object SQLConf { | |||
|
|||
val PARQUET_FILTER_PUSHDOWN_DATE_ENABLED = buildConf("spark.sql.parquet.filterPushdown.date") | |||
.doc("If true, enables Parquet filter push-down optimization for Date. " + | |||
"This configuration only has an effect when 'spark.sql.parquet.filterPushdown' is enabled.") | |||
s"This configuration only has an effect when '${PARQUET_FILTER_PUSHDOWN_ENABLED.key}' is " + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Can we apply the same policy to other places, too?
'spark.sql.optimizer.planChangeLog.level' at L182
spark.sql.parquet.compression.codec
at L397
spark.sql.orc.compression.codec
at L493
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @kiszk . Those instances are impossible. We cannot use the value during defining the configuration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, thanks
Test build #98214 has finished for PR 22877 at commit
|
Merged to master |
Thanks, @kiszk and @dongjoon-hyun |
## What changes were proposed in this pull request? This PR proposes to avoid hardcorded configuration keys in SQLConf's `doc. ## How was this patch tested? Manually verified. Closes apache#22877 from HyukjinKwon/minor-conf-name. Authored-by: hyukjinkwon <gurwls223@apache.org> Signed-off-by: hyukjinkwon <gurwls223@apache.org>
What changes were proposed in this pull request?
This PR proposes to avoid hardcorded configuration keys in SQLConf's `doc.
How was this patch tested?
Manually verified.