一个新项目用 node 做后端,前段时间看到一个删除接口需要操作三张表,大概长这样:
async delete(){
await order.delete();
await item.delete();
await history.delete();
}
然后这个时候从前端调用接口响应时间差不多是 600ms (测试库部署在良心云的便宜实例 docker 上所以比较慢)
由于这几个调用之间没有关联,所以尝试了一下使用 Promise.all()
来提交:
async delete(){
const asyncRes = await Promise.all([
order.delete(),
item.delete(),
history.delete(),
])
}
然后发现响应时间居然直接变成 180ms 了,居然快了这么多。
所以想着有时间就把所有类似的没有关联的请求都换成 all(),但不知道有没有什么大的影响
1
iwh718 2022-11-25 16:39:14 +08:00 via Android
并发吧。
|
2
coderxy 2022-11-25 16:39:17 +08:00
正确的用法就是这样
|
3
ysc3839 2022-11-25 16:43:34 +08:00
await 的话是按顺序执行,一个完成后执行下一个。Promise.all 是同时执行三个,然后等三个都完成。如果没有顺序要求的话是没问题的。
|
4
star7th 2022-11-25 16:45:27 +08:00
没啥不良影响,你可以这么用。
第一种 await 的方式顺序执行,一个执行完成再执行另一个。 第二种方式是,并发执行,等到最后一个结果执行完毕后,再按顺序返回执行结果数据。 一个是顺序执行,一个是并发执行,当然快了很多。 |
5
coldmonkeybit OP @ysc3839 对,我就是想到这点,但就是怕有什么我不知道的地方,才上来确认下,谢谢了。
|
6
coldmonkeybit OP @star7th 明白了,谢谢
|
7
pkoukk 2022-11-25 16:52:51 +08:00 1
不太一样,业务也许可以接受 order 删除了,item 没有成功删除
但不一定能接受 item 删除了,order 没有成功删除 得看你的业务,这种非幂等行为最好还是丢在同个事务里执行 |
8
coldmonkeybit OP @pkoukk 确实,不过假如场景换成查询的话,这种并发的模式感觉挺好用,比多个表关联查询要省事很多,适合我这种不是一上来就做后端的哈哈
|
9
makelove 2022-11-25 18:07:55 +08:00
印象中 mysql 一个 connection 并不能并发查询,因为它并不线程安全
除非你每个查询都用了不同的 connection ,但这样怎么做事务? |
10
gouflv 2022-11-25 18:29:53 +08:00 via iPhone
不需要事务的话,随便
|
11
ysc3839 2022-11-25 18:38:30 +08:00
@makelove 连接库会有处理的吧?假如不支持并发的话,await 的时候另一个请求来了,又去操作数据库,不就出问题了?
|
12
ETiV 2022-11-25 19:22:46 +08:00
可以考虑用 async.queue 控制下并发。
因为你在访问数据库,所以如果并发量没控制好的话,数据库很容易就被你打卦、成单点故障了。。 |
13
zhennann 2022-11-25 19:43:33 +08:00
1. 如果不考虑事务的话,每个 sql 语句的执行都是从连接池中取一个可用的连接,执行完后又把连接释放,并放回连接池。所以,如果并发语句太多,仍然会等待
2. 如果 QPS 少的话,在代码内部使用 Promise.all 可以充分利用连接池。如果 QPS 多的话,Promise.all 价值就不大了。因为连接池总是有限的,不是在这等待,就是在那等待 3. 因此,在业务开发中,考虑到要实现的代码逻辑比较多,都用 Promise.all 做优化,性价比就不高了,还是以提高开发效率优先吧 |
14
stimw 2022-11-26 03:42:29 +08:00 via iPhone
|
15
stimw 2022-11-26 03:46:58 +08:00 via iPhone
@stimw 说明一下,这个提问和楼主问题是有区别的。只有第三个回答( 51 票那个),以及第四个回答的评论区,是跟楼主问题相关的。
|
16
stevezhang 2022-11-26 10:32:45 +08:00
all 你其中有失败的操作不就寄了
|
17
zyronon 2022-11-26 13:29:37 +08:00 1
用 Promise.allSettled()才能拿到所有返回值
|