ReCounters.java returns a counter without retries from create()

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#149 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 1 ngày

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

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
85/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ệ
java
Lĩnh vực
backend

Hướng nghiên cứu

Thay đổi nằm ở create trong src/main/java/co/stateful/retry/ReCounters.java, hiện trả về this.origin.create(name) mà không bọc lại. Hãy so sánh với get, hàm này đã bọc kết quả trong ReCounter, và bọc kết quả của create theo cách tương tự. Việc hoàn tất khi set và incrementAndGet trên một bộ đếm mới tạo thử lại khi gặp IOException, giống các bộ đếm lấy từ get. Yêu cầu không nêu tệp kiểm thử nào, vì vậy hãy kiểm tra xem đã có bài kiểm thử thử lại nào để mở rộng hay chưa.

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

Mô tả

ReCounters.create("hits") returns the original counter without a ReCounter wrapper. Consequently, new ReSttc(sttc).counters().create("hits").set(10L) propagates the first IOException from set, even if a second attempt would succeed. The same counter obtained through get("hits") has retry protection. This contradicts ReSttc's promise to retry requests and also affects incrementAndGet on a newly created counter.

In src/main/java/co/stateful/retry/ReCounters.java, lines 58–60:

public Counter create(final String name) throws IOException {
    return this.origin.create(name);
}

The annotation on create retries creation itself; it cannot intercept later method calls on the returned original counter. get already wraps its result in ReCounter on line 79, whose set and incrementAndGet methods carry @RetryOnFailure.

Wrap the result of create in the same decorator:

public Counter create(final String name) throws IOException {
    return new ReCounter(this.origin.create(name));
}

This one-line change gives counters returned by both entry points the same retry behavior.

Ngôn ngữ chính
Java
Star
10
Fork
3
Merge trung bình
4 giờ 22 phút
Pull request đã merge (30 ngày)
9

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

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: 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

  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 sttc/java-sdk

Tất cả issue của sttc/java-sdk

Issue tương tự

Thêm issue về Java

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.