apache / apache/druid

Clarifying VersionedIntervalTimeline.findFullyOvershadowed()

Ouverte
#8,167 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Improvement Question
Langage dominant
Java
Étoiles
14.1k
Forks
3.8k
Merge moyen
1 j 19 h
PR mergées (30 j)
209

Description

In #7547, @jihoonson added clarifying comments for `completePartitionsTimeline` and `incompletePartitionsTimeline` fields in `VersionedIntervalTimeline`. One of the comments say that "IncompletePartitionsTimeline also includes completePartitionsTimeline".

It explains to me why the second part of the `findFullyOvershadowed()` algorithm iterates only over `incompletePartitionsTimeline`:

https://github.com/apache/incubator-druid/blob/7b9bd5359db80bdd9bb3f547161f94b369469b76/core/src/main/java/org/apache/druid/timeline/VersionedIntervalTimeline.java#L392-L404

But it doesn't explain why then the first part does exactly the same loop over both `completePartitionsTimeline` and `incompletePartitionsTimeline`:

https://github.com/apache/incubator-druid/blob/7b9bd5359db80bdd9bb3f547161f94b369469b76/core/src/main/java/org/apache/druid/timeline/VersionedIntervalTimeline.java#L364-L382

Related to #7433 and #8070.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez dans core/src/main/java/org/apache/druid/timeline/VersionedIntervalTimeline.java et lisez findFullyOvershadowed(), en particulier les boucles autour des lignes 364-404 ainsi que les commentaires de completePartitionsTimeline et incompletePartitionsTimeline. Clarifiez pourquoi la première boucle itère sur les deux timelines, tandis que la seconde itère uniquement sur incompletePartitionsTimeline ; les commentaires devraient rendre cette distinction compréhensible.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
databases
Type d'issue
Documentation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.