alibaba/Sentinel

*bug report* ProcessorSlot的Spi中的isSingleton = false设置不生效

Open

#3,007 创建于 2022年12月29日

在 GitHub 查看
 (4 评论) (0 反应) (0 负责人)Java (23,109 star) (8,150 fork)batch import
area/spigood first issue

描述

Issue Description

Type: bug report

Describe what happened (or what feature you want)

当在非单例模式的ProcessorSlot（A）前加一个单例模式的ProcessorSlot（B）后，所有ProcessorSlotChain中B的next都会同步修改为最新创建的A

Describe what you expected to happen

在上述行为后，不同的ProcessorSlotChain中B的next应该为各自新创建的A，而非共同的A

How to reproduce it (as minimally and precisely as possible)

  1. 添加一个ProcessSlot实现类，添加注解@Spi(order = Spi.ORDER_HIGHEST)

Tell us your environment

sentinel 1.8.3

Anything else we need to know?

DefaultProcessorSlotChain中添加next的方式是直接修改传入的AbstractLinkedProcessorSlot，但是传入的ProcessorSlot有可能是单例或非单例，倘若是单例，则在其后添加next会影响所有的ProcessorSlotChain

贡献者指南

技术栈
java
领域
backend
议题类型
bug
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 hours
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
stale
清晰度议题是否清楚说明期望改动、验收标准和下一步。
mostly clear
前置要求
JavaSentinelSPI
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
30
研究方向
The bug is about ProcessorSlot SPI singleton setting not taking effect when a non singleton ProcessorSlot (A) is preceded by a singleton ProcessorSlot (B). The issue is in DefaultProcessorSlotChain where adding a next node directly modifies the AbstractLinkedProcessorSlot, causing all chains sharing the singleton B to have the same next reference. Investigate the SPI annotation 'isSingleton' handling and the Chain's addLast method. Look at how ProcessorSlotChain constructs and links slots, and propose a fix to properly handle singleton vs non singleton slots when building chains.
*bug report* ProcessorSlot的Spi中的isSingleton = false设置不生效 · alibaba/Sentinel#3007 | Good First Issue