[Bug] Drop branch DDL reports success when the branch directory cannot be deleted
Maintainer thường phản hồi trong vòng 1 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
Hướng nghiên cứu
Tìm FileSystemBranchManager.dropBranch và kiểm tra cách fileIO.delete(branchPath, true) xử lý IOException. So sánh hành vi của nó với createBranch, renameBranch, fastForward và CatalogBranchManager.dropBranch, sau đó tìm các bài kiểm thử branch hoặc DDL liên quan. Hoàn tất khi việc xóa thư mục branch thất bại được truyền đến caller và DDL DROP BRANCH không còn báo thành công.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Search before asking
- I searched in the issues and found nothing similar.
Paimon version
master
Compute Engine
JavaAPI / Flink / Spark (DDL)
Minimal reproduce step
FileSystemBranchManager.dropBranch catches the IOException thrown by fileIO.delete(branchPath, true), logs it at INFO level, and returns normally. Because the method returns void and throws nothing, the drop-branch DDL reports success even though the branch directory and all of its data are still on the filesystem.
Run ALTER TABLE t DROP BRANCH b (filesystem catalog) where the branch directory delete fails, for example due to a permission or filesystem error.
What doesn't meet your expectations?
The DDL succeeds, so the user believes the branch is gone, but its directory and data remain and storage is not reclaimed. A later CREATE BRANCH b with the same name then fails with "Branch name 'b' already exists", which is hard to explain to someone who was told the drop succeeded. The INFO message even says "Please try again", yet no error reaches the caller to prompt a retry.
When the branch directory cannot be deleted, the DDL should fail so the caller knows the branch was not dropped. This also matches createBranch, renameBranch, and fastForward in the same class, and the catalog-backed CatalogBranchManager.dropBranch, which all propagate the failure.
Anything else?
No response
Are you willing to submit a PR?
- I'm willing to submit a PR!
- Ngôn ngữ chính
- Java
- Star
- 3.4k
- Fork
- 1.4k
- Merge trung bình
- 1 ngày 11 giờ
- Pull request đã merge (30 ngày)
- 502
Chuẩn bị môi trường
Chúng tôi chưa kiểm tra các tệp thiết lập môi trường của dự án này. Hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của apache/paimon
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
REST Catalog alterTable silently reports success when the server returns a non-table/column 404Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug] AppendTableITCase.testCompactionInStreamingMode is flaky and can time out under CI loadĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của apache/paimon
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
refinedmods/refinedstorage2#1414 · 1 bình luận ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
In Java's `LongBoundedSum`, setting `lower` to `Long.MIN_VALUE` under-estimates the sensitivityĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
google/differential-privacy#489 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
ScyllaDB Manual: 3 broken linksĐang mởlink-check link-check:manual
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100