Skip to content

Attach retry/backoff outcome to remote-read observations #1045

Description

@Matt711

HttpRetryPolicy/RetryOutcome (cpp/include/kvikio/detail/http_retry.hpp) already computes per-attempt retry decisions and exponential backoff inside remote_handle.cpp's retry loop. None of it (attempt count, backoff time, final HTTP status) reaches Observation or Summary. Relevant to the cloud push specifically: a query that looks slow from max_concurrent_io_tasks's vantage point could actually be S3 throttling triggering retries, which no current metric distinguishes from ordinary latency. Add attempt_count/final_status fields to Observation, threaded from HttpRetryPolicy::evaluate().

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions