apache / apache/druid

Clarifying VersionedIntervalTimeline.findFullyOvershadowed()

Abierto
#8,167 1 comentario 0 reacciones 0 asignados Ver en GitHub
Improvement Question
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.