Clarifying VersionedIntervalTimeline.findFullyOvershadowed()
- Lenguaje dominante
- Java
- Estrellas
- 14.1k
- Forks
- 3.8k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 209
Descripción
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.
Guía de contribución
Línea de trabajo
Comienza en core/src/main/java/org/apache/druid/timeline/VersionedIntervalTimeline.java y lee findFullyOvershadowed(), especialmente los bucles alrededor de las líneas 364-404 y los comentarios de completePartitionsTimeline e incompletePartitionsTimeline. Aclara por qué el primer bucle itera sobre ambas timelines, mientras que el segundo itera solo sobre incompletePartitionsTimeline; los comentarios deberían hacer comprensible esta distinción.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- databases
- Tipo de issue
- Documentación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100