记一次子线程Thread.sleep影响主线程的坑

背景

在实现一个springboot日志记录的aop方法
但是为了防止日志记录的时候占用太多时间影响数据返回，所以采取了多线程记录日志的方法。

我的程序中有个定时任务，每个十分钟把redis中的数据取出来存到mysql中用来持久化，

个人背景（我只是对于多线程有过学习，但是从来没有亲自上手使用过多线程在业务代码上，对于多线程的理解十分片面）

我担心程序在读取完并存取的过程中有新的日志存到redis中会被误删，所以进行了简易的加锁操作

代码如下

public void saveLogFromRedisToMysql() {
        if(redisUtils.hasKey(MhtLogKey)){
            if(redisUtils.hasKey(logLockKey)){//有锁加一
                redisUtils.incr(logLockKey,1);
            }
            else{//没锁创造锁
                redisUtils.set(logLockKey,1);
            }
            List<Object> logs = redisUtils.lGet(MhtLogKey,0,-1);
            List<AdminLog> adminLogs = new ArrayList<>();
            for(Object object:logs){
                AdminLog adminLog = (AdminLog) object;
                adminLogs.add(adminLog);
            }
            boolean flag = adminLogDao.batchSaveLogs(adminLogs);
            if(flag){
                redisUtils.del(MhtLogKey);
            }
            //释放锁
            redisUtils.decr(logLockKey,1);
        }
    }

然后在添加日志到redis中的代码中添加一个判断即可

public void pushMhtLogList(AdminLog adminLog) {
        while (true){
            //如果有锁就循环
            if(redisUtils.hasKey(logLockKey) && (redisUtils.get(logLockKey)).equals(1)){
                System.out.println("存入log等待释放锁中");
                try {
                    Thread.sleep(1000);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
            }
            else{
                break;
            }
        }
        redisUtils.lPush(MhtLogKey,adminLog);
    }

问题

这时候就有问题了，我时使用的@Async来实现多线程

 @Async("asyncServiceExecutor")
    void saveLoginLog(String username, HttpServletRequest request, Long beginTime) {

	****省去细节代码***

	redisService.pushMhtLogList(adminLog);
    }

我理所应当的认为这个函数是不在主线程运行的，那么他调用的redisService.pushMhtLogList(adminLog);
应该也是在另一个线程中运行的，不会影响主线程的运行结果

但是事与愿违

成功的让我的主线程无法返回结果

解决方法

经过几次试错之后
我在
redisService.pushMhtLogList
上加上了
@Async(“asyncServiceExecutor”)
就成功运行了

问题分析

根据我浅显的多线程知识分析，我认为
可能原因是
首先redisService.pushMhtLogList虽然实在子线程中被调用，但是自身不是一个多线程程序，因此在redisService.pushMhtLogList中使用Thread.sleep方法，导致了Thread指向的是主线程，因此主线程等待

不过又感觉不怎么对
最蛋疼的事我发现idea还不方便调试多线程（或许说根本调不了，因为我试了下调不了，百度也没搜索到正确的调试姿势，后来用的土办法log输出来调试的）

应该花点时间去研究一下@Async具体的实现原理，为啥在有@Async的函数中调用另一个函数会影响主线程。

效果展示

去请求

成功请求到数据

但是存log线程还卡着