[ROCKETMQ-306] Specify thread's name of DefaultMQProducerImpl's checkExecutor

• Type: Improvement
• Status: Closed
• Priority: Critical
• Resolution: Invalid
• Affects Version/s: 4.1.0-incubating
• Component/s: rocketmq-client
• Assignee: Xiaorui Wang
• Reporter: acoder2014
• Created: 28/Oct/17 07:03
• Updated: 11/Nov/17 03:13
• Resolved: 11/Nov/17 03:13

Description

Current the way of initializing checkExecutor didn't specify a threadFactory with provided a meaningful thread name which is very helpful to trace the error information,so we better assign a name of this thread pool.

        this.checkExecutor = new ThreadPoolExecutor(
            producer.getCheckThreadPoolMinSize(),
            producer.getCheckThreadPoolMaxSize(),
            1000 * 60,
            TimeUnit.MILLISECONDS,
            this.checkRequestQueue);

Issue Links

• links to: GitHub Pull Request #187