Skip to content

Harden count error handling#89

Open
johncmerfeld wants to merge 3 commits intomainfrom
count-error-handling
Open

Harden count error handling#89
johncmerfeld wants to merge 3 commits intomainfrom
count-error-handling

Conversation

@johncmerfeld
Copy link
Copy Markdown
Contributor

Runway has encountered issues fetching records from ODSs. It's hard to debug those issues because the count method, which runs before fetch, does not handle errors the same way that other methods do.

The two changes here:

  1. handle 401s – we have seen those in the wild so I believe diagnosability would improve if we could at least attempt a token refresh, and I see no cost to doing so
  2. Include the error text in the console output / results file. Again I see no cost to doing this, and adding more verbose errors elsewhere has reaped benefits.

@johncmerfeld johncmerfeld requested a review from tomreitz May 1, 2026 16:43
@johncmerfeld johncmerfeld self-assigned this May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant