Skip to content

Conversation

@ruanwenjun
Copy link
Member

@ruanwenjun ruanwenjun commented Dec 12, 2025

Purpose of the pull request

When server is unhealthy, we will cat the server log, but the server log path is incorrect.

+ cat /home/runner/ds_test/worker-server/logs/dolphinscheduler-master.log
cat: /home/runner/ds_test/worker-server/logs/dolphinscheduler-master.log: No such file or directory

Brief change log

Change the log path when cat

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 contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

@ruanwenjun ruanwenjun requested a review from SbloodyS as a code owner December 12, 2025 02:46
@github-actions github-actions bot added the CI&CD label Dec 12, 2025
@ruanwenjun ruanwenjun added this to the 3.4.0 milestone Dec 12, 2025
@sonarqubecloud
Copy link

Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants