alibaba/Sentinel

提供maxContextNameSize设置选项，对于超出限量的资源优先清理空闲

Open

#2 108 ouverte le 1 avr. 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Java (8 150 forks)batch import
good first issuekind/enhancement

Métriques du dépôt

Stars
 (23 109 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

EntranceNode和DefaultNode构建的invocation tree 会一直存在内存中，因此如果资源大于默认2000，将无法再进行限流。

这一块虽然没有那么重要，但是可以采用这种优化

  • 当数额未超过MAX_CONTEXT_NAME_SIZE，正常处理
  • 当数额超过MAX_CONTEXT_NAME_SIZE，清理 intervalInMs 内无记录的node让出空间。

另外MAX_CONTEXT_NAME_SIZE也可以考虑提供出来供使用者调节。

这样处理在正常情况下应该对性能不会有什么影响，因为一般没有那么多资源限流。而如果真正遇到了需要限流的资源太多，也能够进行处理，而不是直接报错。不过那种场景可能会导致频繁new EntranceNode。而这也可以通过调节MAX_CONTEXT_NAME_SIZE来达到平衡。

Originally posted by @zidoshare in https://github.com/alibaba/Sentinel/discussions/2097#discussioncomment-551580

Guide contributeur

Direction de recherche
Examinez comment EntranceNode et DefaultNode gèrent l'arborescence des appels, identifiez où MAX CONTEXT NAME SIZE est utilisé, et implémentez une propriété configurable ainsi qu'une logique de nettoyage pour les nœuds inactifs lorsque la limite est dépassée.
Stack technique
java
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaSentinel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65