rust-server-sdk icon indicating copy to clipboard operation
rust-server-sdk copied to clipboard

Warning "We recommend blocking no longer than 60 seconds" without an explanation

Open samscott89 opened this issue 1 year ago • 1 comments

Small nit/documentation question.

We set wait_for_initialization to 120s as a conservative estimate of how long to wait for the client to connect. In production we've observed init taking as long as 25s, so this seemed like a reasonable time. We're now getting warnings from the client.

Two minor requests: (a) It would have been nice for this to be documented in the API (b) Can you expand on why we shouldn't wait for longer? For us LD is on the critical path and if we cannot initialize we need to fallback to an offline mode which we would rather avoid.

For completeness, repro is:

let initialized = client.wait_for_initialization(std::time::Duration::from_secs(120)).await;

samscott89 avatar Jul 26 '24 14:07 samscott89