Flaky: a_retry_waits_on_its_registration_and_another_backup_is_refused gets TableAlreadyExistsException

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#951 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 5 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust
Lĩnh vực
databases, testing

Hướng nghiên cứu

Bắt đầu trong crates/engine/src/backup/tests.rs tại backup::tests::a_retry_waits_on_its_registration_and_another_backup_is_refused; issue xác định vấn đề thời gian chập chờn quanh claim_restore_delay và assertion gần dòng 19469. Thay khoảng chờ cố định bằng một cổng do bài kiểm thử điều khiển, giữ đăng ký đầu tiên mở cho đến khi yêu cầu khôi phục thứ hai được gửi. Chạy bài kiểm thử này và xác nhận rằng nó luôn trả về TableInUseException cho yêu cầu khôi phục thứ hai.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

ci:stability

backup::tests::a_retry_waits_on_its_registration_and_another_backup_is_refused in crates/engine/src/backup/tests.rs fails some CI runs with TableAlreadyExistsException("Table already exists: MusicRestored").

Runs that failed:

  • main at 89b94274e, run 42935162912, run-tests (stable), 2026-11-11 00:55 UTC. The panic is at tests.rs:19469, the assertion on the restore of the second backup. The Test runs of main at bf5ea8fc1 and 672d07c6e passed.
  • #900 at 91684d9e4, run 42950822755, run-tests (stable), 2026-11-11 03:13 UTC, with the same panic and message. run-tests (1.88.0) failed in the same run.

The test holds the first restore's registration for 400 ms (claim_restore_delay). It expects a restore of another backup into the same table name to answer TableInUseException while that registration is in flight. The failing runs answered TableAlreadyExistsException, the answer for a target that exists. That fits a second request that arrives after the first registration ended. The test then depends on the timing of the runner.

Proposed fix: hold the first registration until the test sends the second restore. A gate that the test opens can replace the fixed delay.

Ngôn ngữ chính
Rust
Star
517
Fork
60
Merge trung bình
5 ngày 21 giờ
Pull request đã merge (30 ngày)
110

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của ExtendDB/extenddb

Tất cả issue của ExtendDB/extenddb

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.