请注意,你的 Pulsar 集群可能有删除数据的风险
2024-01-09 08:52:21 软件 211观看
摘要在上一篇 Pulsar3.0新功能介绍中提到,在升级到 3.0 的过程中碰到一个致命的问题,就是升级之后 topic 被删除了。正好最近社区也补充了相关细节,本次也接着这个机会再次复盘一下,毕竟这是一个非常致命的 Bug。图片现象先来

在上一篇 Pulsar3.0新功能介绍中提到,在升级到 3.0 的过程中碰到一个致命的问题,就是升级之后 topic 被删除了。UHp28资讯网——每日最新资讯28at.com

正好最近社区也补充了相关细节,本次也接着这个机会再次复盘一下,毕竟这是一个非常致命的 Bug。UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

现象

先来回顾下当时的情况:升级当晚没有出现啥问题,各个流量指标、生产者、消费者数量都是在正常范围内波动。UHp28资讯网——每日最新资讯28at.com

事后才知道,因为只是删除了很少一部分的 topic,所以从监控中反应不出来。UHp28资讯网——每日最新资讯28at.com

早上上班后陆续有部分业务反馈应用连不上 topic,提示 topic nof found.UHp28资讯网——每日最新资讯28at.com

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'Producer': Invocation of init method failed; nested exception is org.apache.pulsar.client.api.PulsarClientException$TopicDoesNotExistException: Topic Not Found.

因为只是部分应用在反馈,所以起初怀疑是 broker 升级之后导致老版本的 pulsar-client 存在兼容性问题。UHp28资讯网——每日最新资讯28at.com

所以我就拿了平时测试用的 topic 再配合多个老版本的 sdk 进行测试,发现没有问题。UHp28资讯网——每日最新资讯28at.com

直到这一步还好,至少证明是小范故障。UHp28资讯网——每日最新资讯28at.com

因为提示的是 topic 不存在,所以就准备查一下 topic 的元数据是否正常。UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

查询后发现元数据是存在的。UHp28资讯网——每日最新资讯28at.com

之后我便想看看提示了 topic 不存在的 topic 的归属,然后再看看那个 broker 中是否有异常日志。UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

发现查看归属的接口也是提示 topic 不存在,此时我便怀疑是 topic 的负载出现了问题,导致这些 topic 没有绑定到具体的 broker。UHp28资讯网——每日最新资讯28at.com

于是便重启了 broker,结果依然没有解决问题。UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

之后我们查询了 topic 的 internal state 发现元数据中会少一个分区。UHp28资讯网——每日最新资讯28at.com

紧急恢复

我们尝试将这个分区数恢复后,发现这个 topic 就可以正常连接了。UHp28资讯网——每日最新资讯28at.com

于是再挑选了几个异常的 topic 发现都是同样的问题,恢复分区数之后也可以正常连接了。UHp28资讯网——每日最新资讯28at.com

所以我写了一个工具遍历了所有的 topic,检测分区数是否正常,不正常时便修复。UHp28资讯网——每日最新资讯28at.com

void checkPartition() {      String namespace = "tenant/ns";      List<String> topicList = pulsarAdmin.topics().getPartitionedTopicList(namespace);      for (String topic : topicList) {          PartitionedTopicStats stats = pulsarAdmin.topics().getPartitionedStats(topic, true);          int partitions = stats.getMetadata().partitions;          int size = stats.getPartitions().size();          if (partitions != size) {              log.info("topic={},partitinotallow={},size={}", topic, partitions, size);              pulsarAdmin.topics().updatePartitionedTopic(topic, partitions);          }     }}

排查

修复好所有 topic 之后便开始排查根因,因为看到的是元数据不一致所以怀疑是 zk 里的数据和 broker 内存中的数据不同导致的这个问题。UHp28资讯网——每日最新资讯28at.com

但我们查看了 zookeeper 中的数据发现一切又是正常的,所以只能转变思路。UHp28资讯网——每日最新资讯28at.com

之后我们通过有问题的 topic 在日志中找到了一个关键日志:UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

以及具体的堆栈。UHp28资讯网——每日最新资讯28at.com

此时具体的原因已经很明显了,元数据这些自然是没问题;根本原因是 topic 被删除了,但被删除的 topic 只是某个分区,所以我们在查询 internalState 时才发发现少一个 topic。UHp28资讯网——每日最新资讯28at.com

通过这个删除日志定位到具体的删除代码:UHp28资讯网——每日最新资讯28at.com

org.apache.pulsar.broker.service.persistent.PersistentTopic#checkReplication

图片图片UHp28资讯网——每日最新资讯28at.com

原来是这里的  configuredClusters 值为空才导致的 topic 调用了 deleteForcefully()被删除。UHp28资讯网——每日最新资讯28at.com

而这个值是从 topic 的 Policy 中获取的。UHp28资讯网——每日最新资讯28at.com

复现问题

通过上图中的堆栈跟踪,怀疑是重启  broker 导致的 topic unload ,同时 broker 又在构建 topic 导致了对 topicPolicy 的读写。UHp28资讯网——每日最新资讯28at.com

最终导致 topicPolicy 为空。UHp28资讯网——每日最新资讯28at.com

只要写个单测可以复现这个问题就好办了:UHp28资讯网——每日最新资讯28at.com

@Test    public void testCreateTopicAndUpdatePolicyConcurrent() throws Exception {        final int topicNum = 100;        final int partition = 10;        // (1) Init topic        admin.namespaces().createNamespace("public/retention");        final String topicName = "persistent://public/retention/policy_with_broker_restart";        for (int i = 0; i < topicNum; i++) {            final String shadowTopicNames = topicName + "_" + i;            admin.topics().createPartitionedTopic(shadowTopicNames, partition);        }        // (2) Set Policy        for (int i = 90; i < 100; i++) {            final String shadowTopicNames = topicName + "_" + i;            CompletableFuture.runAsync(() -> {                while (true) {                    PublishRate publishRate = new PublishRate();                    publishRate.publishThrottlingRateInMsg = 100;                    try {                        admin.topicPolicies().setPublishRate(shadowTopicNames, publishRate);                    } catch (PulsarAdminException e) {                    }                }            });        }        for (int i = 90; i < 100; i++) {            final String shadowTopicNames = topicName + "_" + i;            CompletableFuture.runAsync(() -> {                while (true) {                    try {                        admin.lookups().lookupPartitionedTopic(shadowTopicNames);                    } catch (Exception e) {                    }                }            });        }        admin.namespaces().unload("public/retention");        admin.namespaces().unload("public/retention");        admin.namespaces().unload("public/retention");        Thread.sleep(1000* 5);        for (int i = 0; i < topicNum; i++) {            final String shadowTopicNames = topicName + "_" + i;            log.info("check topic: {}", shadowTopicNames);            PartitionedTopicStats partitionedStats = admin.topics().getPartitionedStats(shadowTopicNames, true);            Assert.assertEquals(partitionedStats.getPartitions().size(), partition);        }    }

同时还得查询元数据有耗时才能复现:UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

只能手动 sleep 模拟这个耗时UHp28资讯网——每日最新资讯28at.com

具体也可以参考这个 issue https://github.com/apache/pulsar/issues/21653#issuecomment-1842962452UHp28资讯网——每日最新资讯28at.com

此时就会发现有 topic 被删除了,而且是随机删除的,因为出现并发的几率本身也是随机的。UHp28资讯网——每日最新资讯28at.com

图片图片UHp28资讯网——每日最新资讯28at.com

这里画了一个流程图就比较清晰了,在 broker 重启的时候会有两个线程同时topicPolicy 进行操作。UHp28资讯网——每日最新资讯28at.com

在 thread3 读取 topicPolicy 进行判断时,thread2 可能还没有把数据准备好,所以就导致了 topic 被删除。UHp28资讯网——每日最新资讯28at.com

修复

图片既然知道了问题原因就好修复了,我们只需要把 thread3 和 thread2 修改为串行执行就好了。UHp28资讯网——每日最新资讯28at.com

这也是处理并发最简单高效的方法,就是直接避免并发;加锁、队列啥的虽然也可以解决,但代码复杂度也高了很多,所以能不并发就尽量不要并发。UHp28资讯网——每日最新资讯28at.com

但要把这个修复推送到社区上游主分支最好是要加上单测,这样即便是后续有其他的改动也能保证这个 bug 不会再次出现。UHp28资讯网——每日最新资讯28at.com

之后在社区大佬的帮助下完善了单测,最终合并了这个修复。UHp28资讯网——每日最新资讯28at.com

再次证明写单测往往比代码更复杂,也更花费时间。UHp28资讯网——每日最新资讯28at.com

PR:https://github.com/apache/pulsar/pull/21704UHp28资讯网——每日最新资讯28at.com

使用修复镜像

因为社区合并代码再发版的周期较长,而我们又急于修复该问题;不然都不敢重启 broker,因为每重启一次都可能会导致不知道哪个 topic 就被删除了。UHp28资讯网——每日最新资讯28at.com

所以我们自己在本地构建了一个修复的镜像,准备在线上进行替换。UHp28资讯网——每日最新资讯28at.com

此时坑又来了,我们满怀信心的替换了一个镜像再观察日志发现居然还有删除的日志

本文链接:http://www.28at.com/showinfo-26-58958-0.html请注意,你的 Pulsar 集群可能有删除数据的风险

声明:本网页内容旨在传播知识,不代表本站观点,若有侵权等问题请及时与本网联系,我们将在第一时间删除处理。

显示全文

上一篇:一文搞定JMM核心原理

下一篇:依赖Kafka的Go单元测试例解

最新热点