yugabyte/yugabyte-db

[DocDB] Add metrics around MemTracker::GcTcmalloc

Open

#17 375 ouverte le 16 mai 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)C (1 003 forks)batch import
area/docdbgood first issuekind/enhancementpriority/medium

Métriques du dépôt

Stars
 (8 229 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (92 PRs mergées en 30 j)

Description

Jira Link: DB-6564

Description

We should add some metrics around how many times we call it and how much memory we release at a time. This is primarily useful for the tserver, but could also be valuable for PG, as we now use a MemTracker in PG and we've observed, in some benchmarks, that the PG time spent on the MemTracker can add up to 5% total time to the scan queries. cc @spolitov @timothy-e

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.

Guide contributeur

Direction de recherche
Examinez le code MemTracker dans src/yb/util/mem tracker.h et .cc pour comprendre comment GcTcmalloc est invoqué. Regardez les implémentations de métriques existantes dans les bases de code tserver et PG. Le problème mentionne l'ajout de compteurs pour le nombre d'appels et la mémoire libérée. Consultez le Jira lié (DB 6564) pour un contexte supplémentaire. Étant donné que le problème a un assigné, coordonnez vous avec lui ou vérifiez s'il y a des travaux en cours.
Stack technique
cpp
Domaine
backenddatabase
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.
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
C++Memory managementMetrics systems
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40