一个排序引发的BUG (中)

简介: 一个排序引发的BUG (中)

咋回事啊?


到底是咋回事呢?

且听我给你分析一波。

上一小节我说了,问题出在排序算法上。

org.apache.dubbo.common.extension.support.ActivateComparator

来,一起看一下:

微信图片_20220428142723.png


首先标号为 ① 的地方就是把 before、after、order 封装了一下,然后提供了几个比较的方法。你知道 ActivateInfo 这个实体里面有这些东西就行了,后面的代码会用到。

然后说说标号为 ② 的地方。

这个地方你别看挺长的,但是其实逻辑特别简单,当前对比的两个 filter 中的任何一个配置了 before、after 就会进入到标号为 ② 的部分的逻辑。

然后这里面的一坨逻辑是的这样的:

image.png


具体逻辑不细说了,等会给你来个直观的演示。

最后标号为 ③ 这个地方,有点意思,稍微多说几句。

能走到标号为 ③ 的地方,说明当前对比的两个 filter 都没有配置 after、before 这两个属性。

直接对比 Order 就行了。

这个地方对 Order 相等的情况还做了一个特殊处理:

o1.getSimpleName().compareTo(o2.getSimpleName()) > 0 ? 1 : -1

如果 Order 相等,再比较类名。

这样做的原因也是保证排序的稳定性。

举个例子,比如这两 Filter,都没有指定 Order:

image.png

代码就变成了这样:

if (a1.order > a2.order) {
    return 1 
} else {
    return -1 
}

简化一下就是这样:

return a1.order > a2.order ? 1:-1

那么这一块的代码,整体就会变成这样:


image.png

好的,经过这样的一番改造。

恭喜你,获得了一个老版本的代码:

左边是之前版本的代码,右边是现在 Master 分支的代码:


image.png

为什么会发生变化,必然是有原因的。

看一眼提交记录:

image.png

这次提交指向了编号为 7778 的提交:

https://github.com/apache/dubbo/pull/7778

而这次提交指向了编号为 7757 的 issue:

image.png

https://github.com/apache/dubbo/issues/7757

而这个 issue 在前面提到的编号为 8055 的 issue 里也提到了:

image.png

这个 issue 主要就是两张图。

第一张图是这样的:

在没有任何自定义 Filter,仅有官方原有的 Filter 的情况下,构建出来的 Filter 链,ExecuteLimitFilter 在 MonitorFilter 之前。

image.png

第二张图是这样的:

image.png

在加入了一系列自定义的 Filter(没有指定 Order)之后,ExecuteLimitFilter 就排在了 MonitorFilter 之后了。

至于这两个 Filter 排前排后的影响是什么,和文本关系不大,就不扩展了,你有兴趣的可以去看看对应的链接。

总之,只有这样的判断逻辑是不稳当的:

return a1.order > a2.order ? 1:-1

目录
相关文章
|
7月前
|
消息中间件 前端开发 关系型数据库
🤔️测试问我:为啥阅读量计数这么简单的功能你都能写出bug?
🤔️测试问我:为啥阅读量计数这么简单的功能你都能写出bug?
|
7月前
|
数据可视化 定位技术
学无止境——记录一个被别人发现的bug
学无止境——记录一个被别人发现的bug
学无止境——记录一个被别人发现的bug
|
算法 Python
更正--冒泡排序代码
更正--冒泡排序代码
排列的时候如何避免重复?
排列的时候如何避免重复?
93 0
|
人工智能 算法 搜索推荐
比冒泡算法还简单的排序算法:看起来满是bug的程序,居然是对的
比冒泡算法还简单的排序算法:看起来满是bug的程序,居然是对的
113 0
比冒泡算法还简单的排序算法:看起来满是bug的程序,居然是对的
|
Dubbo 应用服务中间件 测试技术
一个排序引发的BUG (上)
一个排序引发的BUG (上)
109 0
一个排序引发的BUG (上)
|
Dubbo Java 应用服务中间件
一个排序引发的BUG (下)
一个排序引发的BUG (下)
118 0
一个排序引发的BUG (下)
|
关系型数据库 MySQL Java
【BUG日记】【MySQL】多个排序字段,是有优先级的,先来先优先。
【BUG日记】【MySQL】多个排序字段,是有优先级的,先来先优先。
244 0
【BUG日记】【MySQL】多个排序字段,是有优先级的,先来先优先。
IE里面的一些BUG记录
网上已经有很多类似的记录了,这里写这个是给自己在项目中碰到的问题做个简单的记录,以后将持续更新 1.IE67 border-bottom失效 一个a标签,想要使用移上去后会在下面显示一个横条,我用border-bottom做了一个效果,IE6 7必须加了line-height后才能显示,应该是因为没有高度的原因
IE里面的一些BUG记录
|
Web App开发 算法 前端开发
前端 BUG 录 - 因数组排序造成的卡顿
前端 BUG 录 - 因 lodashjs debounce 去抖优化造成的 bug 这两个 BUG 其实是同一个 BUG,怎么说呢?两个都没错,错就错在同时使用了。 因为我没处理边界,导致我会给一个大数组排序,造成了卡顿。然后卡顿造成了节流时间成为了笑话。
147 0
前端 BUG 录 - 因数组排序造成的卡顿