Skip to content

Bump ch to 0.7.1 (via bumping ecto_ch) for improved DB disconnect handling#6024

Merged
zoldar merged 1 commit intomasterfrom
bump-ecto-ch
Jan 26, 2026
Merged

Bump ch to 0.7.1 (via bumping ecto_ch) for improved DB disconnect handling#6024
zoldar merged 1 commit intomasterfrom
bump-ecto-ch

Conversation

@zoldar
Copy link
Contributor

@zoldar zoldar commented Jan 26, 2026

ch uses new capability added to db_connection (plausible/ch#292) for graceful handling of DB disconnects.

@zoldar zoldar added the preview label Jan 26, 2026
@github-actions
Copy link

Preview environment👷🏼‍♀️🏗️
PR-6024

@zoldar zoldar requested review from a team and cnkk January 26, 2026 08:31
Copy link
Contributor

@apata apata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What metric would we be looking at to see if it has an effect?

@zoldar
Copy link
Contributor Author

zoldar commented Jan 26, 2026

@apata Hopefully less or virtually no context-less errors like "Ch.Connection (#PID<0.5417.0>) disconnected: ** (Mint.TransportError) socket closed" in our logs. I'm doing the same for persistor, where it seems to cause more issues.

@zoldar zoldar added this pull request to the merge queue Jan 26, 2026
Merged via the queue into master with commit 2175b32 Jan 26, 2026
19 checks passed
@zoldar zoldar deleted the bump-ecto-ch branch January 26, 2026 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants