Clarifying VersionedIntervalTimeline.findFullyOvershadowed()
- 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