You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2021/01/19 16:25:03 UTC

[GitHub] [zeppelin] Reamer commented on a change in pull request #4028: [ZEPPELIN-5210] Upgrade Configuration library to version 2

Reamer commented on a change in pull request #4028:
URL: https://github.com/apache/zeppelin/pull/4028#discussion_r560307125



##########
File path: docs/setup/operation/configuration.md
##########
@@ -24,12 +24,14 @@ limitations under the License.
 <div id="toc"></div>
 
 ## Zeppelin Properties
-There are two locations you can configure Apache Zeppelin.
 
-* **Environment variables** can be defined `conf/zeppelin-env.sh`(`conf\zeppelin-env.cmd` for Windows).
-* **Java properties** can be defined in `conf/zeppelin-site.xml`.
+Zeppelin can be configured via several sources.
+
+Sources descending by priority:
+ - system properties
+ - environment variables can be defined `conf/zeppelin-env.sh`(`conf\zeppelin-env.cmd` for Windows).
+ - configuration file can be defined in `conf/zeppelin-site.xml`

Review comment:
       You are right. I will change the order to prevent breaking changes.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org