Skip to content

fix: default connection_validation to false — per-checkout SELECT NUL…#60

Open
Esity wants to merge 2 commits into
mainfrom
fix/disable-connection-validation-default
Open

fix: default connection_validation to false — per-checkout SELECT NUL…#60
Esity wants to merge 2 commits into
mainfrom
fix/disable-connection-validation-default

Conversation

@Esity

@Esity Esity commented Jun 11, 2026

Copy link
Copy Markdown

…L validation degrades throughput

Query-time error handling already recovers stale connections. Also remove inline || 600 / || 14_400 pool timeout fallbacks that shadowed the documented settings defaults.

What

Brief description of the change.

Why

What problem does this solve or what feature does it add?

Checklist

  • Tests pass (bundle exec rspec)
  • RuboCop passes (bundle exec rubocop)
  • CHANGELOG.md updated (if user-facing change)
  • No new security concerns introduced

…L validation degrades throughput

Query-time error handling already recovers stale connections. Also remove
inline || 600 / || 14_400 pool timeout fallbacks that shadowed the
documented settings defaults.
@Esity Esity self-assigned this Jun 11, 2026
@Esity Esity requested a review from a team as a code owner June 11, 2026 03:41
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