Zac

Results 3 comments of Zac

> 这个集群 + raft 的配置中心模式,为了提升整体的 tps,不需要走水平发布通知了,因为 raft 本身的舒服同步就包括了这一步走,因此这个 ConfigCacheService 本身的逻辑转移到了 raft 状态机当中去了 实验过,k8s部署三个pod的集群,使用内置数据库,通过批量删除接口删除配置,服务还能通过缓存读到已删除的配置信息,但是通过接口已经查不到了,注释掉判断条件再试就没问题了

> EmbeddedConfigDumpApplyHook 这里针对批量删除的场景有bug, ![image](https://private-user-images.githubusercontent.com/20452676/329532519-391c3e91-41ed-4560-8d52-0ad2e2763b47.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTUzOTA3OTEsIm5iZiI6MTcxNTM5MDQ5MSwicGF0aCI6Ii8yMDQ1MjY3Ni8zMjk1MzI1MTktMzkxYzNlOTEtNDFlZC00NTYwLThkNTItMGFkMmUyNzYzYjQ3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA1MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNTExVDAxMjEzMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTU5NmNjNWQ2YjllNzY3MTQ5YmUxMzRiNDJmZDAzZDg0MGZkZWIyM2M4MmFiMDI1OTYzMTVkOTQ3MzlkOTI3OTUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.BtJStxaoGPGM6kmJmxp9BRjXJHAiVVEFiWuDh14FhOk) > > @ZrBac 你本地测试下,把这个问题修复掉是不是就好了,如果可以的话,提交个PR 已验证,确实为该bug导致

> > > EmbeddedConfigDumpApplyHook 这里针对批量删除的场景有bug, ![image](https://private-user-images.githubusercontent.com/20452676/329532519-391c3e91-41ed-4560-8d52-0ad2e2763b47.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTUzOTA3OTEsIm5iZiI6MTcxNTM5MDQ5MSwicGF0aCI6Ii8yMDQ1MjY3Ni8zMjk1MzI1MTktMzkxYzNlOTEtNDFlZC00NTYwLThkNTItMGFkMmUyNzYzYjQ3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA1MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNTExVDAxMjEzMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTU5NmNjNWQ2YjllNzY3MTQ5YmUxMzRiNDJmZDAzZDg0MGZkZWIyM2M4MmFiMDI1OTYzMTVkOTQ3MzlkOTI3OTUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.BtJStxaoGPGM6kmJmxp9BRjXJHAiVVEFiWuDh14FhOk) > > > @ZrBac 你本地测试下,把这个问题修复掉是不是就好了,如果可以的话,提交个PR > > > > > > 已验证,确实为该bug导致 > > ok,你是否可以提交一个pr修复下呢 已经提交了