Trino 并发查询卡死问题排查

背景

目前实现了一个异步读的需求,大概逻辑就是在构造pageSource的时候,就把读取数据的请求发送出去,然后把数据缓存在内存中。并根据异步读的完成情况,设置这个pageSource的block状态。这样在调用getNextPage的时候,线程就不会因为读取数据而被阻塞住,可以直接从缓存中读取数据。提高线程的利用率。

为了防止并发场景下多个查询读取同一个stripe的时候,会出现重复去读相同数据的场景,造成资源浪费。我们用一个static caffeine cache来保存stripe信息,key是对应的stripe, value是一个queue, 保存每个查询对应的future对象。

在每个stripe读之前,会判断这个cache里面是不是已经有这个key了,如果有说明其他查询已经开始异步读这个stripe, 那这个查询则无需再异步读,直接把这个查询的future放入value中,等第一个查询完成了异步读,则会把value里所有的future取出来并设置为完成状态。这样所有读取这个stripe的查询都完成该stripe的异步读。

问题现象及分析

现象

当并发跑多个查询,且这几个查询读取的数据相同时,会偶发的出现个别查询卡死现象

分析

通过查看Trino webUI,看到失败的查询是在某个worker上卡住导致超时的

进到对应worker里面去搜该查询的split finished日志,因为每个split结束时,都会打印相应的信息,从而可以筛选出卡住的那个split

再根据这个split去查对应的日志,我会给每个stripe在结束异步读时都打印对应日志,进而找到是哪个stripe一直卡住没有完成,因为这个stripe一直没有完成,就会导致读取数据一直不会结束,进而导致查询超时失败。

找到卡住的stripe后,查找对应日志, query2发现该stripe已经有query1正在异步读了,因此把query2的future放入了cache的value中。而query1完成该stripe异步读时,打印cache中value的size值为1,说明在并发情况下,query2把自己的future放入队列,而query1获取队列时并没有感知到。两个操作之间有gap。

解决

为了解决写入和读取之间的gap, 其实就是要保证读写之间的原子性。之前的代码实现中读就是从cache中拿value, 然后遍历value中的future设置完成,写就是从cache中拿value, 然后往里加数据。读和写的过程中拿value的操作和处理value的操作不具备原子性,因此中间是存在并发问题的。

有问题的代码:

java 复制代码
//写入
cache.get(stripeIdentifier, s -> {  
    return new ConcurrentLinkedDeque<>();  
}).add(readFinishedSignal);

//读取
ConcurrentLinkedDeque<CompletableFuture> targetStripe = cache.getIfPresent(stripeIdentifier);  
if (targetStripe != null) {  
    targetStripe.forEach(finishSignal -> finishSignal.complete(null));  
    cache.invalidate(stripeIdentifier);  
}

caffeine cache 可以使用asMap()方法拿到cache内部的concurrentHashMap。然后读和写使用compute和computeIfPresent方法,因为在这两个方法里面的代码是保证原子性的,而且使用这两个方法的时候是会加锁的,这样就可以保证放入队列中的future是能别的线程拿到的。

改正后的代码:

java 复制代码
//写入
cache.asMap().compute(stripeIdentifier, (k, v) -> {  
    if (v == null) {  
        ConcurrentLinkedDeque<CompletableFuture> deque = new ConcurrentLinkedDeque<>();  
        deque.add(readFinishedSignal);  
        return deque;  
    } else {  
        v.add(readFinishedSignal);  
        return v;  
    }  
});

//读取
cache.asMap().computeIfPresent(stripeIdentifier, (k, v) -> {  
    v.forEach(finishSignal -> finishSignal.complete(null));  
    return null;  
});

改正后再测试,确实没有再出现并发卡死的现象了。

相关推荐
jinan8868 分钟前
企业的移动终端安全怎么管理?
大数据·网络·安全·数据分析·开源软件
叶辰 .1 小时前
ES使用聚合aggregations实战(2025.04.02更新)
大数据·elasticsearch·jenkins
cwtlw1 小时前
Spring相关面试题总结
java·笔记·后端·spring
zxsz_com_cn1 小时前
风电行业预测性维护解决方案:AIoT驱动下的风机健康管理革命
大数据·运维·人工智能
yiridancan1 小时前
深入浅出:Spring Bean 的初始化流程详解
java·开发语言·后端·spring
Vitalia7 小时前
从零开始学Rust:枚举(enum)与模式匹配核心机制
开发语言·后端·rust
飞飞翼8 小时前
python-flask
后端·python·flask
草捏子9 小时前
最终一致性避坑指南:小白也能看懂的分布式系统生存法则
后端
一个public的class9 小时前
什么是 Java 泛型
java·开发语言·后端
说私域10 小时前
基于开源AI大模型与S2B2C模式的线下服务型门店增长策略研究——以AI智能名片与小程序源码技术为核心
大数据·人工智能·小程序·开源