Test a_retry_waits_on_its_registration_and_another_backup_is_refused fails when the first restore finishes before the third request

Abierto Apto para principiantes
  • ¿Algo no está bien?
#908 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 4 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
76/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
testing

Línea de trabajo

Empieza por crates/engine/src/backup/tests.rs, especialmente por la prueba que falla y las pruebas cercanas que usan bulk_load_gate y closed_gate(). Mantén la carga masiva de la primera restauración bloqueada hasta después de la aserción de la tercera solicitud y, luego, abre la compuerta. Ejecuta backup::tests::a_retry_waits_on_its_registration_and_another_backup_is_refused; debería pasar y la tercera solicitud debería recibir TableInUseException.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug

What happens

backup::tests::a_retry_waits_on_its_registration_and_another_backup_is_refused in crates/engine/src/backup/tests.rs failed in the mongodb-rust job of #830's CI on 2026-11-07 (run 42325897438, job 128260989427):

panicked at crates/engine/src/backup/tests.rs:18576:5:
TableAlreadyExistsException("Table already exists: MusicRestored")

The same job passed twice on main at 76b9710dc and once on #411. #830 does not change crates/engine.

Why

The test sends three restore requests for the target MusicRestored:

  1. claim_restore_delay holds the first request's registration for 400 ms.
  2. A retry of the same request waits for that registration and gets the CREATING description.
  3. A restore of another backup into the same name expects TableInUseException. A committed registration gives that answer while its restore job runs.

Nothing holds the first restore's job, and the backup has two items. If the job finishes before the third request, the table is ACTIVE, and the third request gets TableAlreadyExistsException. That is the answer in the failed run.

Suggested fix

Hold the first restore's bulk load with bulk_load_gate and a closed_gate(), as other tests in the same file do. Open the gate after the assertion on the third request.

Related: #740.

Lenguaje dominante
Rust
Estrellas
517
Forks
60
Merge medio
5 d 3 h
PR fusionados (30 d)
79

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de ExtendDB/extenddb

Todos los issues de ExtendDB/extenddb

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.