yugabyte/yugabyte-db

[DocDB] IntentAwareIterator::Next() does not advance intent iterator

Open

#22.605 aberto em 29 de mai. de 2024

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)C (1.003 forks)batch import
area/docdbgood first issuekind/bugpriority/low

Métricas do repositório

Stars
 (8.229 stars)
Métricas de merge de PR
 (Mesclagem média 17d 21h) (92 fundiu PRs em 30d)

Description

Jira Link: DB-11515

Description

Current implementation of the method:

void IntentAwareIterator::Next() {
  VLOG_WITH_FUNC(4);

  if (!status_.ok()) {
    return;
  }

  if (IsEntryRegular()) {
    SkipFutureRecords<Direction::kForward>(iter_.Next());
    FillEntry();
  }
}

There is no logic for the intent_iter_, which seems wrong as it should be moved next in some cases.

Currently, the method is used only from GetHelperBase::Prepare() for the row getter initialization. And it does not lead to any inconsistency, but gives an extra decoding (i.e. for a packed record or delete record) and maybe one extra Seek/Next call.

Issue Type

kind/bug

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

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

Guia do colaborador

Direção de pesquisa
Examine o método IntentAwareIterator::Next() e entenda quando intent iter deve ser avançado. Rastreie o uso em GetHelperBase::Prepare() para confirmar o comportamento atual e identificar a lógica ausente.
Pilha de tecnologia
ccpp
Domain
databasebackend
Tipo Issue
Bug
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Database internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50