dragonflydb/dragonfly

refactor DbSlice::ExpireParams

Chiusa

#3634 aperta il 3 set 2024

Vedi su GitHubGuida contributor
 (13 commenti) (0 reazioni) (1 assegnatario)C++ (1177 fork)batch import
good first issue

Metriche repository

Star
 (30.465 stelle)
Metriche merge PR
 (Merge medio 2g 11h) (201 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la struttura DbSlice::ExpireParams attuale nei file sorgente per comprendere come vengono utilizzati i campi 'value', 'unit' e 'absolute'. Quindi, sostituisci questi campi con un singolo 'int64 t ms timestamp' calcolato nel costruttore o in un metodo. Assicurati di aggiornare tutti i riferimenti e che i test passino. Rivedi eventuali commenti o discussioni collegate per ulteriore contesto.
Tech stack
cpp
Dominio
database
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Bloccata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++ basicsFamiliarity with DbSlice
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60