疑问：客户端任务执行的触发没有用线程池

[LeYunone]

版本

所有版本

问题

在客户端触发定时任务的动作里，见 JobThread 类；

if (triggerParam.getExecutorTimeout() > 0) {
    // limit timeout
    Thread futureThread = null;
    try {
        FutureTask<Boolean> futureTask = new FutureTask<Boolean>(new Callable<Boolean>() {
            @Override
            public Boolean call() throws Exception {

                // init job context
                XxlJobContext.setXxlJobContext(xxlJobContext);

            handler.execute();
            return true;
            }
        });
        futureThread = new Thread(futureTask);
        futureThread.start();

        Boolean tempResult = futureTask.get(triggerParam.getExecutorTimeout(), TimeUnit.SECONDS);
	}
//.....

采用了创建线程为载体，实现任务超时的回调函数；

疑问

因为需要考虑任务上下文 XxlJobContext ，父线程和子线程需要共享一个 ThreadLocal ，所以使用线程+InheritableThreadLocal 的组合（上述代码）可以理解；
不过目前存有大量任务，都设有超时时间，并且存在会同时频繁执行的情况；
这样一来以上代码对原本平稳运行的系统会带来不小的隐患。
所以未来是否会考虑使用线程池+TransmittableThreadLocal 的模式取代上述旧的执行逻辑 🤝

[cdknowz]

我猜测作者觉得每个任务都是同等重要的吧，如果超时时间设置过长，会导致占用线程池过久，导致后面任务被放入队列阻塞。

[LeYunone]

我猜测作者觉得每个任务都是同等重要的吧，如果超时时间设置过长，会导致占用线程池过久，导致后面任务被放入队列阻塞。

很赞同这个猜测，是选择使用线程池接受任务出现先后顺序阻塞，还是采用原创建线程方式由系统性能兜底，最好由使用者自行选择一类；（对xxl-job二开中在此添加了一个字段区分任务的执行方式）