-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix(storage): invalid storage details cache after writing operations #2018
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
base: main
Are you sure you want to change the base?
Conversation
…enaming/copying/removing/puting Signed-off-by: hcrgm <hcrgm@qq.com>
…an error Signed-off-by: hcrgm <hcrgm@qq.com>
| details, err, _ := detailsG.Do(storage.GetStorage().MountPath, func() (*model.StorageDetails, error) { | ||
| ret, err := wd.GetDetails(ctx) | ||
| if err != nil { | ||
| Cache.InvalidateStorageDetails(storage) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
此时Cache中还没有storage的容量信息,该语句产生的效果一定是什么也不做
| if storage.Config().NoCache { | ||
| return nil, nil | ||
| } | ||
| Cache.InvalidateStorageDetails(storage) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
绝大多数情况下创建文件夹、移动、重命名都不会影响空间占用,不需要删除缓存
| Cache.linkCache.DeleteKey(stdpath.Join(dstKey, srcRawObj.GetName())) | ||
| } | ||
| if !storage.Config().NoCache { | ||
| Cache.InvalidateStorageDetails(storage) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上
| Cache.linkCache.DeleteKey(stdpath.Join(dirKey, dstName)) | ||
| } | ||
| if !storage.Config().NoCache { | ||
| Cache.InvalidateStorageDetails(storage) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上
Description / 描述
对存储驱动写操作成功后(上传、删除等),清除存储驱动的详情缓存,使容量数据更实时、准确。
Motivation and Context / 背景
发生上传、删除等写入操作后,容量信息没有发生变化,需要重新加载存储、根目录刷新或等缓存自然过期才发生变化。
How Has This Been Tested? / 测试
Checklist / 检查清单
我已阅读 CONTRIBUTING 文档。
go fmtor prettier.我已使用
go fmt或 prettier 格式化提交的代码。我已为此 PR 添加了适当的标签(如无权限或需要的标签不存在,请在描述中说明,管理员将后续处理)。
我已在适当情况下使用"Request review"功能请求相关代码作者进行审查。
我已相应更新了相关仓库(若适用)。