Skip to content

[issues/766] fix: 模型同步一直停留在lfs_start的问题#1037

Open
shaoyou520 wants to merge 1 commit intoOpenCSGs:mainfrom
shaoyou520:main
Open

[issues/766] fix: 模型同步一直停留在lfs_start的问题#1037
shaoyou520 wants to merge 1 commit intoOpenCSGs:mainfrom
shaoyou520:main

Conversation

@shaoyou520
Copy link
Copy Markdown

@shaoyou520 shaoyou520 commented Apr 29, 2026

What is this feature?
fix:解决issues 766的问题,对于一些分支检查添加状态的更新

[Add a brief description of what the feature or update does.]
Why do we need this feature?
使功能更加的完善,处理问题如下:
1:worker 的map 退出时未清理
2:worker 的context 没有及时cancel, 只有重试如果是相同task时才 调用cancel
3:一些查询不到时的状态更改

[Add a description of the problem the feature is trying to solve.]

Who is this feature for?

[Add information on what kind of user the feature is for.]

Which issue(s) does this PR fix?:

Fixes # issue 766

Special notes for your reviewer:

@shaoyou520 shaoyou520 force-pushed the main branch 3 times, most recently from bc02c75 to 1ed036f Compare April 29, 2026 08:04
@shaoyou520 shaoyou520 changed the title [issues/766] fix: 模型同步一直停留在lfs_start的问题&& /mirror/sync 接口添加运行中的检查 [issues/766] fix: 模型同步一直停留在lfs_start的问题 Apr 29, 2026
Copy link
Copy Markdown
Collaborator

@pulltheflower pulltheflower left a comment

Choose a reason for hiding this comment

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

🥇 非常感谢您的贡献 👍 ,代码中有一些改动的点和我们之前的设计有一点不同,我们可以讨论看下怎么优化更好

Comment thread mirror/lfssyncer/lfs.go Outdated
Comment thread mirror/lfssyncer/lfs.go Outdated
Comment thread mirror/manager/lfs_worker_manager.go Outdated
Comment thread mirror/manager/lfs_worker_manager.go
Comment thread mirror/manager/lfs_worker_manager.go Outdated
@shaoyou520 shaoyou520 force-pushed the main branch 4 times, most recently from 25b24a0 to 2beb1b6 Compare April 30, 2026 01:58
shaoyou520

This comment was marked as duplicate.

Comment thread mirror/lfssyncer/lfs.go Outdated
Comment thread mirror/lfssyncer/lfs.go
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.

2 participants