Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug][Master,Worker] The task execution path should be calculated by the worker, not the master(任务执行路径应该交给worker去计算,而不是master) #4716

Closed
zhanguohao opened this issue Feb 7, 2021 · 0 comments · Fixed by #4717
Labels
bug Something isn't working
Milestone

Comments

@zhanguohao
Copy link
Contributor

**For better global communication, Please describe it in English. If you feel the description in English is not clear, then you can append description in Chinese(just for Mandarin(CN)), thx! **
Describe the bug
If the basedir(common.properties/data.basedir.path) positions of the two nodes are not the same, the execution path calculated by the Master is inconsistent with the actual running path of the Worker, which will cause the operation to fail.
如果两个节点basedir(common.properties/data.basedir.path)位置不相同,Master 计算的执行路径与Worker实际运行路径不一致时导致运行失败

image

To Reproduce
There are two node ,install path is different

Expected behavior

  • Master does not participate in the calculation of the actual execution path and log path. The execution path and log path should be handed over to the Worker for calculation and reported to the master, and the master just update the database.
  • Master不参与计算实际的执行路径和日志路径,执行路径和日志路径应该交给Worker去计算,并且上报到master,Master更新数据库即可

Which version of Dolphin Scheduler:

  • [1.3.3]
  • [1.3.4]
@CalvinKirs CalvinKirs added the bug Something isn't working label Feb 11, 2021
@davidzollo davidzollo added this to the 1.3.6-release milestone Feb 12, 2021
davidzollo pushed a commit that referenced this issue Feb 12, 2021
…d by the worker, not the master (#4717)

* [fix][api,mater,worker] 修复master 计算实际执行路径,worker上报实际执行路径

* remove unused import
lenboo pushed a commit that referenced this issue Mar 18, 2021
…ld be calculated by the worker, not the master #4717 (#5073)

issue #4716
pr #4717
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants