apache / apache/druid

Clarifying VersionedIntervalTimeline.findFullyOvershadowed()

Aperta
#8,167 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Improvement Question
Lingua principale
Java
Stelle
14.1k
Fork
3.8k
Merge medio
2g 58m
PR unite (30g)
233

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia da core/src/main/java/org/apache/druid/timeline/VersionedIntervalTimeline.java e leggi findFullyOvershadowed(), in particolare i cicli intorno alle righe 364-404 e i commenti per completePartitionsTimeline e incompletePartitionsTimeline. Chiarisci perché il primo ciclo itera su entrambe le timelines, mentre il secondo itera solo su incompletePartitionsTimeline; i commenti dovrebbero rendere comprensibile questa distinzione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
databases
Tipo di issue
Documentazione
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.