eclipse-paho / eclipse-paho/paho.mqtt.java

ExecutorService can be used instead of ScheduledExecutorService

Open
#603 9 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
2.3k
Forks
919
PR merge metrics
No merged PRs in 30d

Description

The MqttAsyncClient needs a ScheduledExecutorService param for the constructor but only the functionality of a ExecutorService is needed. Other frameworks works more often with ExecutorService and provides good functionality to create such a service. Based on all that points ExecutorService should be used instead.

Contributor guide

Open the contributing guide

Research direction

Start at the MqttAsyncClient constructor and trace every use of its ScheduledExecutorService parameter. Check the existing tests and public API expectations before assessing the change; done means the client needs only ExecutorService functionality without breaking supported construction paths.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.