Skip to content

Conversation

@nPraml
Copy link
Collaborator

@nPraml nPraml commented Mar 18, 2025

Hello @rbygrave,

During development, we encountered an issue where an exception occurred during container creation, and the connection completion escalated into an infinite loop.
Stacktrace:

	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnConnectionForceClose(ConnectionPool.java:545)
	at io.ebean.datasource/io.ebean.datasource.pool.PooledConnection.close(PooledConnection.java:516)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.testConnection(ConnectionPool.java:397)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnTheConnection(ConnectionPool.java:563)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnConnectionForceClose(ConnectionPool.java:545)
	at io.ebean.datasource/io.ebean.datasource.pool.PooledConnection.close(PooledConnection.java:516)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.testConnection(ConnectionPool.java:397)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnTheConnection(ConnectionPool.java:563)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnConnectionForceClose(ConnectionPool.java:545)
	at io.ebean.datasource/io.ebean.datasource.pool.PooledConnection.close(PooledConnection.java:516)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.testConnection(ConnectionPool.java:397)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnTheConnection(ConnectionPool.java:563)
	at io.ebean.datasource/io.ebean.datasource.pool.ConnectionPool.returnConnectionForceClose(ConnectionPool.java:545)

We have implemented a proposed fix in this PR.

Can you check it please?

@rbygrave rbygrave added this to the 9.3 milestone Mar 19, 2025
@rbygrave rbygrave merged commit 0ff4468 into ebean-orm:master Mar 19, 2025
1 check passed
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