apache / dolphinscheduler

Apache DolphinScheduler is the modern data orchestration platform. Agile to create high performance workflow with low-code
https://dolphinscheduler.apache.org/
Apache License 2.0
12.73k stars 4.58k forks source link

[Improvement-16458] Add a executable permission to the scripts #16459

Closed JinsYin closed 1 month ago

JinsYin commented 1 month ago

Purpose of the pull request

close #16458 Add a executable permission to the scripts.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

JinsYin commented 1 month ago

@SbloodyS When we download binary packages and deploy them, we can use the bin/start.sh command to start the service without executing the bash bin/start.sh command. This is a common practice in most projects, such as start-dfs.sh, stop-all.sh, etc. in Hadoop

SbloodyS commented 1 month ago

@SbloodyS When we download binary packages and deploy them, we can use the bin/start.sh command to start the service without executing the bash bin/start.sh command. This is a common practice in most projects, such as start-dfs.sh, stop-all.sh, etc. in Hadoop

We do not recommend that users use these scripts directly. There are specific descriptions of starting and stopping using dolphinscheduler-daemon.sh in official website. https://dolphinscheduler.apache.org/en-us/docs/3.2.2/guide/installation/pseudo-cluster

So this modification is meaning less. Closing for now.