-
Notifications
You must be signed in to change notification settings - Fork 63
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
v7.9.7 上传问题 #32
Comments
@suconghou 请问你是用什么 API 上传文件的?我们最近确实修改了表单上传的逻辑,将文件内容完整加载到本地内存,表单上传用于 4M 以下小文件,一般不会导致内存不够的问题。大文件推荐使用分片上传,最近也没有什么变更。 |
@bachue 我们的主要场景是10M - 20M 的高清图片和一些 100M 左右的小视频,之前一直也是用表单上传,运行良好,只升级了一下版本,服务就崩了. |
几十兆的东西没必要非得用分片上传,反而更麻烦. |
@suconghou v7.9.7 之前的版本中,表单上传在特殊场景存在小问题,为了修复这个问题需要加载表单信息到内存,所以会提高内存峰值。所以如果您 SDK 升级到 v7.9.7,强烈建议使用分片上传。 |
分片上传没有 类似 body 没有可预知的准确大小, |
@suconghou |
@YangSen-qn |
@suconghou |
我们正在构建一个文件上传中间服务,期望以流的形式传输,但看到 ioutil.ReadAll 之后很明显与需求不符,为此替换成文件会多一次磁盘io |
从v7.9.5升级到最新 v7.9.7 后, 服务频繁OOM, 查看最近SDK源码后,发现新增了
ioutil.ReadAll
, 将用户上传的数据一次性读入内存. 针对小文件还好,稍大一点的文件就非常占用内存.原先只需要不到100M内存的服务,现在至少需要1G内存. (而且这个策略还额外造成了整体上传的延时)感觉明显是一个倒退,如果是为了重试,希望可以把是否重试的开关交给用户选择.
The text was updated successfully, but these errors were encountered: