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

实例中简单任务出现脑裂异常问题处理 #625

Open
wants to merge 118 commits into
base: 4.3.2-findbug
Choose a base branch
from

Conversation

lvhjean
Copy link

@lvhjean lvhjean commented Apr 24, 2023

fit: #620
解决 PowerJob 4.3.2 的samples中实例,因执行任务客户机返回结果,在任务Worker的address更新到instance库之前返回,导致InstanceManager更新时获取不到instance库的TrackerAddress的问题;

@KFCFans
Copy link
Member

KFCFans commented May 7, 2023

我在你的 issue 回复问题的原因了,其实 server 的逻辑是没有问题的,所以不能去调整这个顺序。
事实上这个顺序也是精心设计的。
server 为了保证“至少调度一次”,一定需要先执行调度,再写表。这样假如调度失败,无法完成写表,就会有后续的容灾机制触发重调度。

假如先写了表,此事系统重启或者异常导致调度未执行,那状态就出错了。

KFCFans and others added 30 commits February 8, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants