Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connection pool #9

Merged
merged 1 commit into from
Nov 20, 2024
Merged

Connection pool #9

merged 1 commit into from
Nov 20, 2024

Conversation

eaneto
Copy link
Owner

@eaneto eaneto commented Nov 6, 2024

Creates a connection pool so that connections can be reused instead of
creating new connections. The pool has a fixed size, at start ekilibri
will try to establish the same number of connections in the
configuration file, errors are simply logged. The health check process
tries checks if the servers are healthy, when the server is unhealthy
all connections are dropped and the pool goes to "reconnect" state, in
this state every call to get a connection will establish a new
connection. When the server is considered healthy again, remaining
connections are dropped and new ones are established.

The command server was changed to use axum instead of a custom server,
this was done so that the tests run against a correct(already validated)
HTTP implementation, so bugs in ekilibri's implementation were caught
this way.

Creates a connection pool so that connections can be reused instead of
creating new connections. The pool has a fixed size, at start ekilibri
will try to establish the same number of connections in the
configuration file, errors are simply logged. The health check process
tries checks if the servers are healthy, when the server is unhealthy
all connections are dropped and the pool goes to "reconnect" state, in
this state every call to get a connection will establish a new
connection. When the server is considered healthy again, remaining
connections are dropped and new ones are established.

The command server was changed to use axum instead of a custom server,
this was done so that the tests run against a correct(already validated)
HTTP implementation, so bugs in ekilibri's implementation were caught
this way.
@eaneto eaneto marked this pull request as ready for review November 20, 2024 22:30
@eaneto eaneto merged commit 8e93200 into main Nov 20, 2024
1 check passed
@eaneto eaneto deleted the connection-pool branch November 20, 2024 22:43
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