[apolloconfig/apollo]发现有request中已经编码实现了按key全局搜索配置的功能,但是没有merge到mater,理由是?

2024-07-05 492 views
5

我们系统需要实现按照key来全局搜索配置的功能,查看以往的issues,发现这个已经实现了该功能,但是一直没有合并到master,请教下,是这个功能实现的有问题还是有其他的考虑呢

实现的分支:https://github.com/apolloconfig/apollo/pull/2724

回答

9

@lepdou 我们再评估下当时这个实现的设计和需求场景?

9

这个需求场景还是很多的。如果一个开发人员具有权限或者管理的应用/namespace很多,有时候找不到我要修改的key在哪个项目里的哪个namespace里。 按照目前的逻辑,得先找到应用、再找到 namespace,路径有点长。 因为大部分修改配置的场景都是针对 key 的修改,所以直接搜索 key,能快速触达。


当时没合进去的原因是担心性能问题。这个我找时间优化一下吧,默认搞个开关,关闭这个功能。用户可以在系统配置里打开这个开关。

1

@hezhaoye 也可以详细描述你一下你的实际业务场景

1

我看了你那个分支的代码,实现是用了sql用了like查询,如果换成=查询,再加上索引,性能应该没问题。这个到时你优化,只是增加一个开关吗,还是实现方式上有其他的优化呢

6

@lepdou @nobodyiam review代码发现一个问题,目前的实现是将各个环境的模糊匹配结果返回到portal端(adminService不分页,这里的数据有可能超过几千行),由portal端进行分页,这样感觉会对adminService的数据库压力很大,为啥不考虑在adminservice进行分页呢