zhongjiajie commented on pull request #7151: URL: https://github.com/apache/dolphinscheduler/pull/7151#issuecomment-995433993
> I wonder if this is a proper change for common usage. The config file `install_config.conf` is used for installation. and not all deployment cases use the `install.sh` for one-click-install. So, reading properties from `install_config.conf` may cause confusion. Hi @kevinjmh, actually we read the configuration in env variable, and `install_config.cnf` is only export variable to env. If user not use `install.sh` to deploy DS, they could use ohter way to export variable to env. But in this case, our document lead user using script `install.sh` to deploy, should export from `install_config.cnf` -- 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]
