Provide a sync + non-blocking API when server-side evaluation is enabled
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- rust
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu bằng cách đọc luồng khởi tạo xung quanh flagsmith.options.enable_local_evaluation, update_environment và eval_context. So sánh các luồng đánh giá cục bộ và đánh giá phía máy chủ, đồng thời xác định các API đồng bộ và không chặn nên cung cấp những gì; hoàn thành nghĩa là đánh giá phía máy chủ không còn yêu cầu eager fetch và đánh giá cục bộ vẫn có thể sử dụng trong cả ngữ cảnh đồng bộ lẫn bất đồng bộ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We'd like to use this SDK in a variety of different services were performance matters. We almost only use it with local side evaluation and in different contexts, sync, async and sometimes in async contexts but with manual future implementation, for example when implementing tower services (https://docs.rs/tower-service/latest/tower_service/).
It appears that if server side evaluation is enabled there is no need to do any fetch eagerly. Cf this related code:
if flagsmith.options.enable_local_evaluation {
// Update environment once...
if let Err(e) = update_environment(&client, &ds, &environment_url) {
log::warn!("Failed to fetch environment on initialization: {}. Will retry in background.", e);
}
with update_environment writing to the the eval_context.
It would be nice if we could split this API between local evaluation and remote evaluation because they do not actually have the same bounds. And local could work in any contexts without sync/async issues.
I'd be happy to discuss this and eventually work on this if this direction is accepted.
- Ngôn ngữ chính
- Rust
- Star
- 8
- Fork
- 8
- Merge trung bình
- 20 giờ 41 phút
- Pull request đã merge (30 ngày)
- 2
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
- Đọ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 Flagsmith/flagsmith-rust-client
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Flagsmith/flagsmith-rust-client#58 · 1 bình luận ·
-
Local evaluation silently drops identity overrides on large environmentsCó thể đã có người làm @marc-merino đã nhận 5 ngày trước. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 55/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 58/100
Flagsmith/flagsmith-rust-client#43 · 1 bình luận ·
Tất cả issue của Flagsmith/flagsmith-rust-client
Issue tương tự
-
Trash, bin or rubbish?Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 63/100
pop-os/cosmic-files#2117 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
L: java:gradle L: java:maven L: ruby:bundler T: bug 🐞
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
dependabot/dependabot-core#16547 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug status:ready-for-test
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
UniClipboard/UniClipboard#1985 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
RustCrypto/SSH#596 ·
-
area:breg rust
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
registrystack/registry-stack#2034 ·
Maintainer thường phản hồi trong vòng 1 ngày