yugabyte/yugabyte-db

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

Open

#22 605 ouverte le 29 mai 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)C (1 003 forks)batch import
area/docdbgood first issuekind/bugpriority/low

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-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.

Guide contributeur

Direction de recherche
Examinez la methode IntentAwareIterator::Next() et comprenez quand intent iter doit etre avance. Tracez l'utilisation dans GetHelperBase::Prepare() pour confirmer le comportement actuel et identifier la logique manquante.
Stack technique
ccpp
Domaine
databasebackend
Type d'issue
Bug
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Database internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50