slackapi / slackapi/java-slack-sdk
5 threads (for the default singleton) can be created even when SlackConfig#statsEnabled is false
Offen
@seratch arbeitet bereits daran.
Seit 19.5.2022.
enhancement
project:slack-api-client
- Vorherrschende Sprache
- Java
- Sterne
- 602
- Forks
- 232
- Ø Merge
- 4 T. 9 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
Developers can disable API clients' metrics management by setting SlackConfig#statsEnabled to false. Even in this case, the default SlackConfig object creation (as a static field in the class) starts 5 threads and those threads are unused. See the repro steps below for details.
Reproducible in:
any versions
The Slack SDK version
any versions
Java Runtime version
any versions
OS info
any versions
Steps to reproduce:
Running the following code and checking the thread dump.
SlackConfig config = new SlackConfig();
config.setStatsEnabled(false);
Slack slack = Slack.getInstance(config);
Expected result:
No threads under a "slack-api-metrics" thread group are created.
Actual result:
5 threads for the default singleton SlackConfig instance are created.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Bewertung
Dieses Issue wurde noch nicht bewertet.