Spring @Scheduled线程模型探究 - 发现问题

最新推荐文章于 2024-07-13 23:37:32 发布

zyc920716

最新推荐文章于 2024-07-13 23:37:32 发布

阅读量276

点赞数

分类专栏： spring 文章标签： spring boot spring java

本文链接：https://blog.youkuaiyun.com/zyc920716/article/details/104442875

版权

spring 专栏收录该内容

3 篇文章

订阅专栏

最近工程中发现一个奇怪的问题，简单代码逻辑如下所示：

@EnableScheduling
@Component
public class SimpleImplement {
    private static final Logger LOG = LoggerFactory.getLogger(SimpleImplement.class);

    private volatile List<String> list = Collections.EMPTY_LIST;

    private final Queue<Thread> waiters = Queues.newConcurrentLinkedQueue();

    // 如果list为空挂起线程
    public List<String> get(){
        while(CollectionUtils.isEmpty(list)){
            try {
                Thread current = Thread.currentThread();
                waiters.add(current);
                LOG.info("list 为空挂起线程 {}.", current.getName());
                LockSupport.park();
            }catch (Exception e){}
        }
        return list;
    }

    // 异步更新list，如果list不为空则唤醒被挂起的线程
    @Scheduled(fixedDelay = 1000)
    private void sync(){
        // 加载数据
        list = ...;
        if(CollectionUtils.isNotEmpty(list)){
            Thread waiter = null;
            while (!waiters.isEmpty()){
                waiter = waiters.remove();
                LOG.info("同步数据 list 不为空唤醒线程 {}.", waiter.getName());
                LockSupport.unpark(waiter);
            }
        }
    }

上边代码对list异步加载，调用get接口的时候如果list为空就挂起当前线程，待有数据加载时再进行唤醒。我们知道加锁也可以实现等待唤醒，这里用LockSupport是因为这里是异步加载数据不存在数据竞争问题，加锁会让多个线程串行执行，影响任务执行效率。

问题
get()接口在工程中好几处被调用，测试发现被挂起的线程有时会唤醒失败。反复测试几次从日志中看被阻塞的线程都是 scheduling-1，看起像是定时器的线程。于是翻看了代码发现几处调用get()接口的地方都采用@Scheduled注解进行周期轮询。

分析
怀疑这里可能是@Scheduled的实现是单线程，几处定时任务都是有同一个线程执行，如果这样调用sync()接口的线程也是调用get()接口的线程，如果被阻塞那么唤醒动作就一直得不到执行。

验证
为了验证我们的想法，写了个简单的测试代码及结果如下:

@Component
@EnableScheduling
public class ScheduledTest {
    private static Logger LOG = LoggerFactory.getLogger(ScheduledTest.class);

    @Scheduled(fixedDelay = 1000)
    public void method1(){
        LOG.info("线程 {} 执行method1 .", Thread.currentThread().getName());
    }

    @Scheduled(fixedDelay = 1000)
    public void method2(){
        LOG.info("线程 {} 执行method2 .", Thread.currentThread().getName());
    }
}