eclipse-paho / eclipse-paho/paho.mqtt.java
ExecutorService can be used instead of ScheduledExecutorService
- 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
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