apache/dubbo

JVMUtil performance improvement

Open

#13 327 ouverte le 7 nov. 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Java (26 453 forks)batch import
component/sdkhelp wantedtype/enhancement

Métriques du dépôt

Stars
 (41 524 stars)
Métriques de merge PR
 (Merge moyen 7j 14h) (20 PRs mergées en 30 j)

Description

https://github.com/apache/dubbo/blob/3.0/dubbo-common/src/main/java/org/apache/dubbo/common/utils/JVMUtil.java

JVMUtil. jstack方法调用整体使用OutputStream作为输出可提升性能，而内部JVMUtil.getThreadDumpString使用String长字符串拼接反而降低了性能。 建议修改内部JVMUtil.getThreadDumpString方法同样为流式处理，减少字符串拼接的性能开销

同时，getThreadDumpString方法内部，可能还考虑性能影响，增加了栈深度控制（默认32）。 这个深度控制、对性能的提升，可能没有流式处理改造能带来的性能提升高。建议应先修改流式处理以提升性能。是否可经测试评估性能影响，取消栈深度控制。以便dubbo的threaddump能够使用在各类Thread分析工具中（如flamegraph.pl等）。

Guide contributeur

Direction de recherche
Modifiez la méthode interne getThreadDumpString pour utiliser OutputStream pour la sortie en streaming au lieu de la concaténation de chaînes. Évaluez si le contrôle de profondeur de pile (par défaut 32) offre encore un avantage de performance suffisant après le passage au streaming, ou s'il peut être supprimé.
Stack technique
java
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaStream API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70