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

Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2022/08/20 11:26:40 UTC

[GitHub] [dolphinscheduler] DarkAssassinator commented on a diff in pull request #11393: [Fix] Fix Environment variable data error

DarkAssassinator commented on code in PR #11393:
URL: https://github.com/apache/dolphinscheduler/pull/11393#discussion_r950684863


##########
.github/workflows/cluster-test/mysql/dolphinscheduler_env.sh:
##########
@@ -15,6 +15,9 @@
 # limitations under the License.
 #
 
+# Load system configuration
+source /etc/profile

Review Comment:
   i think so, and the environment variables do not only exist in the /etc/profile file, I think the specificity of the worker host should be ignored, which can enhance the user's autonomy



-- 
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: commits-unsubscribe@dolphinscheduler.apache.org

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