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

fix(mis): mis-server启动时,不完整同步一次封锁状态 #1374

Merged
merged 5 commits into from
Jul 24, 2024

Conversation

ddadaal
Copy link
Member

@ddadaal ddadaal commented Jul 23, 2024

mis-server启动时,不完整同步一次封锁状态,而是只在后台运行,以减少mis-server启动时间。

UI和API修改:

  • AdminService.UpdateBlockStatus API行为改变:当调用API时后台正在进行一次同步,那么调用此API会返回ALREADY_EXISTS
  • UI上,当用户手动点击同步封锁状态时,如果后台正在进行一次同步,那么会报错

Copy link

changeset-bot bot commented Jul 23, 2024

🦋 Changeset detected

Latest commit: 1ed5a0e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages
Name Type
@scow/grpc-api Patch
@scow/mis-server Patch
@scow/mis-web Patch
@scow/protos Patch
@scow/lib-hook Patch
@scow/lib-operation-log Patch
@scow/lib-scheduler-adapter Patch
@scow/lib-server Patch
@scow/lib-web Patch
@scow/audit-server Patch
@scow/portal-server Patch
@scow/portal-web Patch
@scow/ai Patch
@scow/auth Patch
@scow/gateway Patch
@scow/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ddadaal ddadaal requested a review from tongchong July 23, 2024 15:46
@pkuhpc-review-bot pkuhpc-review-bot bot added the Code-ReviewRequested Code Review Requested label Jul 23, 2024
@ddadaal ddadaal marked this pull request as ready for review July 23, 2024 15:46
@ddadaal ddadaal merged commit c214bd2 into master Jul 24, 2024
9 checks passed
@ddadaal ddadaal deleted the dont-wait-for-block-status-sync branch July 24, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code-ReviewRequested Code Review Requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants