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.

Sign up for GitHub

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

Jump to bottom

[Bug][Server] Wrong excludeFilters cause the worker server to start the master's beans and threads like LowerWeightHostManager and RefreshResourceTask #5176

Closed
chengshiwen opened this issue Mar 30, 2021 · 1 comment · Fixed by #5182 or #5278
Closed

[Bug][Server] Wrong excludeFilters cause the worker server to start the master's beans and threads like LowerWeightHostManager and RefreshResourceTask #5176

chengshiwen opened this issue Mar 30, 2021 · 1 comment · Fixed by #5182 or #5278
Milestone
1.3.6-release

Comments

@chengshiwen
Copy link
Member

chengshiwen commented Mar 30, 2021 •
edited

To Reproduce
Steps to reproduce the behavior, for example:

  1. Add one line logger.info("RefreshResourceTask: {}", workerHostWeights); in RefreshResourceTask#run (Picture 1)
  2. Re-run MasterServer and WorkerServer
  3. See log of MasterServer and WorkerServer
  4. The LowerWeightHostManager and RefreshResourceTask logs of master appear in theWorkerServer log (Picture 2)
  5. Other master's beans and threads will also appear in theWorkerServer like MasterRegistry, ZookeeperNodeManager and etc

Picture 1
image

Picture 2
image

Expected behavior
Bug fixed

Which version of Dolphin Scheduler:
-[1.3.x]
-[dev]
@chengshiwen chengshiwen mentioned this issue Mar 30, 2021
Merged
@chengshiwen
Copy link
Member Author

chengshiwen commented Mar 30, 2021 •
edited

@CalvinKirs Please pay attention to this issue, it can be put into 1.3.6-release. I have fixed this issue in #5122.

@chengshiwen chengshiwen mentioned this issue Mar 30, 2021
Merged
@chengshiwen chengshiwen mentioned this issue Apr 14, 2021
Merged
@chengshiwen chengshiwen mentioned this issue Apr 14, 2021
Merged
@chengshiwen chengshiwen added this to the 1.3.6-release milestone Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.6-release
1 participant
@chengshiwen