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

Disable OracleConnectionTest#testConnectNoLeak tests #1482

Conversation

tsegismont
Copy link
Contributor

Disabled for Oracle because we frequently get ORA-12516 Cannot connect to database

See https://docs.oracle.com/en/error-help/db/ora-12516/?r=23ai

Probable cause is that the service handler gets in a blocked state and is not accepting new connections.

Disabled for Oracle because we frequently get ORA-12516 Cannot connect to database

See https://docs.oracle.com/en/error-help/db/ora-12516/?r=23ai

Probable cause is that the service handler gets in a blocked state and is not accepting new connections.

Signed-off-by: Thomas Segismont <[email protected]>
@tsegismont tsegismont merged commit c30df2f into eclipse-vertx:master Jan 7, 2025
18 checks passed
@tsegismont tsegismont deleted the ignore-test-connect-no-leak-oracle branch January 7, 2025 18:02
tsegismont added a commit to tsegismont/vertx-sql-client that referenced this pull request Jan 8, 2025
tsegismont added a commit that referenced this pull request Jan 9, 2025
* Revert "Disable OracleConnectionTest#testConnectNoLeak tests (#1482)"

This reverts commit c30df2f.

* Reduce number of connections made in testConnectNoLeak tests

We only need a few of them to verify whether there is a leak.
And we avoid putting to much pressure on the Oracle test DB.

Signed-off-by: Thomas Segismont <[email protected]>

---------

Signed-off-by: Thomas Segismont <[email protected]>
tsegismont added a commit to tsegismont/vertx-sql-client that referenced this pull request Jan 9, 2025
* Revert "Disable OracleConnectionTest#testConnectNoLeak tests (eclipse-vertx#1482)"

This reverts commit c30df2f.

* Reduce number of connections made in testConnectNoLeak tests

We only need a few of them to verify whether there is a leak.
And we avoid putting to much pressure on the Oracle test DB.

Signed-off-by: Thomas Segismont <[email protected]>

---------

Signed-off-by: Thomas Segismont <[email protected]>
tsegismont added a commit that referenced this pull request Jan 9, 2025
* Revert "Disable OracleConnectionTest#testConnectNoLeak tests (#1482)"

This reverts commit c30df2f.

* Reduce number of connections made in testConnectNoLeak tests

We only need a few of them to verify whether there is a leak.
And we avoid putting to much pressure on the Oracle test DB.



---------

Signed-off-by: Thomas Segismont <[email protected]>
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