dragonflydb/dragonfly

refactor DbSlice::ExpireParams

Fermée

#3 634 ouverte le 3 sept. 2024

Voir sur GitHubGuide contributeur
 (13 commentaires) (0 réaction) (1 personne assignée)C++ (1 177 forks)batch import
good first issue

Métriques du dépôt

Stars
 (30 465 étoiles)
Métriques de merge PR
 (Merge moyen 2j 11h) (201 PRs mergées en 30 j)

Description

we have several fields in DbSlice::ExpireParams: int64_t value = INT64_MIN; // undefined TimeUnit unit = TimeUnit::SEC; bool absolute = false;

All these fields can be exchanged with one int64_t ms_timestamp, that will be created in the constructor or method.

Guide contributeur

Direction de recherche
Examinez la structure DbSlice::ExpireParams actuelle dans les fichiers sources pour comprendre comment les champs 'value', 'unit' et 'absolute' sont utilisés. Ensuite, remplacez ces champs par un seul 'int64 t ms timestamp' calculé dans le constructeur ou une méthode. Assurez vous de mettre à jour toutes les références et que les tests passent. Consultez les commentaires ou discussions liées pour un contexte supplémentaire.
Stack technique
cpp
Domaine
database
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.
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.
Bloquée
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++ basicsFamiliarity with DbSlice
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60