mfyuce commented on PR #3523: URL: https://github.com/apache/iggy/pull/3523#issuecomment-4762715888
Thanks for the thorough review @ryerraguntla! Both issues addressed in the latest push: **409 / "already exists" on `create_index()`** — `create_index()` now absorbs `409 CONFLICT` and `400 BAD_REQUEST` whose body contains \"already exists\" as `Ok(())` with an `info!` log. Covers the concurrent-open race and the retry-after-succeeded-5xx path. **No timeout on `reqwest::Client`** — Added `request_timeout: Option<String>` to `QuickwitSinkConfig` (default `30s`, configurable via TOML). The raw client is now built with `Client::builder().timeout(request_timeout)`, bounding health probes, `has_index()`, and `create_index()` under network partition. /ready -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
