-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
refactor(fs):Refactor the move function and fix known issues #353
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
Conversation
fix(move):Fix move logic
移动怎么变成任务了,我觉得如果真要实现跨存储移动,不如使用那个给复制任务加一个 callback 的方案 |
一但报错,重试只能从头开始,任务个数限制也相当没了,本地缓存磁盘第一个爆。 |
其实#211 最开始添加的时候就是任务处理方式,我也只是照着思路改,callback方案我会研究一下,再看是否实现 |
我们并不限制用户使用何种方式移动他们的文件,他们也可以选择用复制功能来迁移文件,我们提供的仅作为一种方案,移动功能还会优化,但直接阉割部分功能成为半移动是绝不可取的,如果移动后的源文件还要用户删除,那用户会觉得还不如用复制功能,那功能存在的意义不就没有了? |
没发正式版前在群里问有什么bug我说了,三天没人修都说跨盘移动功能没问题,以为没人修就简单改了先跑通,跨盘移动功能要不要revert你们讨论决定吧。 |
请问一下这个功能有没有开关可以让管理员全局禁用? |
No description provided.