Skip to content

If no suitable host is found, try rechecking the database state.#5953

Open
japm wants to merge 1 commit intonpgsql:mainfrom
japm:main
Open

If no suitable host is found, try rechecking the database state.#5953
japm wants to merge 1 commit intonpgsql:mainfrom
japm:main

Conversation

@japm
Copy link

@japm japm commented Nov 24, 2024

When all hosts does not meet the requirements, try force check database state.

The state of the database connections seems to be put as invalid driven by
message errors/exceptions.

We have a situation where all hosts are invalidated but the databases and pgbouncers
are alive an reachable.

We know that because setting Host Recheck Seconds to zero fixes the issue.

This proposal add a pass to the NpgsqlMultiHostDataSource.Get forcing a database check in case of
none suitable host.

This is a proposal, i can not compile the code ( requires .net 9, requires also some sort of signing procedure)

@japm japm requested review from roji and vonzshik as code owners November 24, 2024 14:05
@NinoFloris
Copy link
Member

Please first open an issue describing what should be fixed. When it comes to this PR, your changes won't work correctly and they cannot be merged.

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.

3 participants