Skip to content

feat: add transient error for PostgreSQL lock timeouts - #716

Merged
wschurman merged 1 commit into
mainfrom
10-05-feat_add_transient_error_for_postgresql_lock_timeouts
Oct 5, 2026
Merged

wschurman merged 1 commit into
mainfrom
10-05-feat_add_transient_error_for_postgresql_lock_timeouts

Conversation

@wschurman

@wschurman wschurman commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Why

Re-implementation of #714 on top of #715.

How

Prompt:

add new error for pg code 55P03, use PR 714 as reference but a new error class

Test Plan

CI

wschurman commented Oct 5, 2026 •

Copy link
Copy Markdown
Member Author

@codecov

codecov Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (6de0405) to head (abfd2a1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##              main      #716    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          110       110            
  Lines        18499     18523    +24     
  Branches      1597       955   -642     
==========================================
+ Hits         18499     18523    +24     
Flag Coverage Δ
integration 30.72% <83.33%> (+0.06%) ⬆️
unittest 94.95% <100.00%> (+0.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

wschurman commented Oct 5, 2026 •

Copy link
Copy Markdown
Member Author

Merge activity

  • Oct 5, 5:05 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Oct 5, 5:07 PM UTC: Graphite rebased this pull request as part of a merge.
  • Oct 5, 5:08 PM UTC: @wschurman merged this pull request with Graphite.

@wschurman
wschurman changed the base branch from 10-05-feat_move_knex-specific_database_adapter_errors_into_entity-database-adapter-knex to graphite-base/716 October 5, 2026 17:05
@wschurman
wschurman changed the base branch from graphite-base/716 to main October 5, 2026 17:05
@wschurman
wschurman force-pushed the 10-05-feat_add_transient_error_for_postgresql_lock_timeouts branch from f5e9d63 to abfd2a1 Compare October 5, 2026 17:06
@wschurman
wschurman merged commit 13ce74e into main Oct 5, 2026
4 checks passed
@wschurman
wschurman deleted the 10-05-feat_add_transient_error_for_postgresql_lock_timeouts branch October 5, 2026 17:08
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.

2 participants