反对使用Spring封装的多线程类原因

网友投稿 632 2022-10-05

反对使用Spring封装的多线程类原因

反对使用Spring封装的多线程类原因

目录1.Spring的异步代码2.是SpringBoot救了你3.End

前言:

工作总难免会遇到被故障所驱使,其实是开启了线程池的暴力使用模式

我有必要简单的复述一下。其主要原因,就是开发人员,在每一次方法调用里,都创建了一个单独的线程池去处理。这样的话,如果请求量一增加,整个操作系统的压力就会耗尽,最终所有的业务都无法响应。

我一直认为这是一个非常偶发的低级错误,发生频率非常的低。但随着这样的故障越来越多,xjjdog认识到这是一个普遍的现象。

以异步性能优化为目bfYuebKNac的,反而带来的整体业务不可用的结果,是非常打脸的一种优化。

1.Spring的异步代码

Spring作为java届的杠把子框架,其过度封装的API深得开发人员的喜爱。根据语义化编程的逻辑,只要某些关键字在语言层面上过得去,我们就可以把它给加上去。比如@Async注解。

我永远想不通是什么给了开发人员勇气,去加上这个@Async注解,因为这种涉及到多线程的东西,即使是自己去创建线程,也是心怀敬畏,唯恐扰了操作系统的安宁。@Async这样的黑盒,真的可以那么顺畅的使用么?

我们不妨debug一下代码,让子弹飞一会儿。

首先,生成一个小小的项目,然后在主类上加上必须的注解。嗯,别忘了这一环,否则你后面加的注解将没什么用处。

@SpringBootApplication

@EnableAsync

publicclassDemoApplication{

创造一个带@Async注解的方法。

@Component

publicclassAsyncService{

@Async

publicvoidasync(){

try{

Thread.sleep(1000);

System.out.println(Thread.currentThread());

}catch(Exceptionex){

ex.printStackTrace();

}

}

}

然后,做一个对应的test接口,访问时会调用这个async方法。

@ResponseBody

@GetMapping("test")

publicvoidtest(){

service.async();

}

访问时,直接打个断点,即可获取执行异步线程的线程池。

可以看到,异步任务使用了一个线程池,它的corePoolSize=8, 阻塞队列采用了无界队列LinkedBlockingQueue。一旦采用了这样的组合,最大线程数就会形同虚设,因为超出8个线程的任务,将全部会被放到无界队列里。使得下面的代码变成了摆设。

thrownewTaskRejectedException("Executor["+executor+"]didnotaccepttask:"+task,var4);

如果你的访问量非常大,这些任务将全部堆积在LinkedBlockingQueue里。情况好一点的,这些任务的执行会变得延迟很大;情况坏一点的,任务太多将直接造成内存溢出OOM!

你可能会说,我可以自己指定另外一个ThreadPoolExceute,然后使用@Async注解来声明啊。说这话的同学,一定是能力比较强,或者Review的代码比较少,没有经过猪队友的洗礼。

2.是SpringBoot救了你

SpringBoot是个好东西。

在TaskExecutionAutoConfiguration中,通过生成ThreadPoolTaskExecutor的Bean,来提供默认的Executor。

@ConditionalOnMissingBean({Executor.class})

publicThreadPoolTaskExecutorapplicationTaskExecutor(TaskExecutorBuilderbuilder){

returnbuilder.build();

}

也就是我们上面所说的那个。如果没有SpringBoot的助力,Spring将默认使用SimpleAsyncTaskExecutor。

参见org.springframework.aop.interceptor.AsyncExecutionInterceptor。

@Override

@Nullable

protectedExecutorgetDefaultExecutor(@NullableBeanFactorybeanFactory){

Executohttp://rdefaultExecutor=super.getDefaultExecutor(beanFactory);

return(defaultExecutor!=null?defaultExecutor:newSimpleAsyncTaskExecutor());

}

这就是Spring大仙所干的事。

SimpleAsyncTaskExecutor类设计的非常操蛋,因为它每执行一次,都会创建一个单独的线程,根本没有共用线程池。比如你的TPS是1000,异步执行了任务,那么你每秒将会生成1000个线程!

这明显是想要累死操作系统的节奏。

protectedvoiddoExecute(Runnabletask){

Threadthread=(this.threadFactory!=null?this.threadFactory.newThread(task):createThread(task));

thread.start();

}

3.End

明眼人一看,这种使用new线程的处理方式将会是非常可怕的。但就拿Spring本身来说,引用SimpleAsyncTaskExecutor这个类的地方还不少,包括比较流行的AsyncRestTemplate。

这暴露了很多风险,尤其是竟然在这些列表中看到了redis的身影。这个类的设计,使得任务的执行变的非常的不可控。

看这个API,我感觉Spring是进入了设计的魔怔状态。

这个东西的隐藏bug可能还会更深!比如org.springframework.context.event.EventListener注解,用于实现DDD那套所谓的事件驱动模式,有不少框架直接set了AsyncRestTemplate,那么就等死吧。

赶紧把SimpleAsyncTaskExecutor加入你的API黑名单,或者埋坑清单吧!

创建线程有那么难么?需要使用Spring创建的线程?有时候我实在是想不通,暴露出这样的接口目的是为了什么。

就连原生的线程池我们还没搞明白呢,你还给包了一层,这是方便我们甩锅啊!

版权声明:本文内容由网络用户投稿,版权归原作者所有,本站不拥有其著作权,亦不承担相应法律责任。如果您发现本站中有涉嫌抄袭或描述失实的内容,请联系我们jiasou666@gmail.com 处理,核实后本网站将在24小时内删除侵权内容。

上一篇:小程序之如何让swiper组件来实现轮播图的功能(vue使用swiper做轮播图)
下一篇:MongoDB 数据模型
相关文章

 发表评论

暂时没有评论,来抢沙发吧~