yaooqinn commented on code in PR #4410: URL: https://github.com/apache/kyuubi/pull/4410#discussion_r1118272978
########## docs/deployment/settings.md: ########## @@ -122,9 +122,18 @@ You can configure the Kyuubi properties in `$KYUUBI_HOME/conf/kyuubi-defaults.co ## Kyuubi Configurations # -# kyuubi.authentication NONE Review Comment: We may give a much more practical example. The current example is from the template directly, where all examples for individual configs are commented out, while we shall give meaningful ones on the contrary. Or, for legible, we could delete this entire block here. For example, `You can configure the Kyuubi properties in $KYUUBI_HOME/conf/kyuubi-defaults.conf, see $KYUUBI_HOME/conf/kyuubi-defaults.conf.template as an example,. ` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
