dragonflydb/dragonfly

refactor DbSlice::ExpireParams

Fechada

#3.634 aberto em 3 de set. de 2024

Ver no GitHubGuia do colaborador
 (13 comentários) (0 reação) (1 responsável)C++ (1.177 forks)batch import
good first issue

Métricas do repositório

Stars
 (30.465 estrelas)
Métricas de merge de PR
 (Mesclagem média 2d 11h) (201 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine a estrutura DbSlice::ExpireParams atual nos arquivos fonte para entender como os campos 'value', 'unit' e 'absolute' são usados. Em seguida, substitua esses campos por um único 'int64 t ms timestamp' que é calculado no construtor ou em um método. Certifique se de atualizar todas as referências e que os testes passem. Revise quaisquer comentários ou discussões vinculadas para contexto adicional.
Pilha de tecnologia
cpp
Domain
database
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++ basicsFamiliarity with DbSlice
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60