Skip to content
New issue

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

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

Already on GitHub? # to your account

Add github workflow. #195

Merged
merged 1 commit into from
Dec 10, 2020
Merged

Add github workflow. #195

merged 1 commit into from
Dec 10, 2020

Conversation

jameshilliard
Copy link
Contributor

No description provided.

@@ -327,8 +327,11 @@ public synchronized void close() throws HibernateException
{
if (session != null)
{
session.close();
session = null;
synchronized (DB.class)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is to guard against this error I was seeing:

org.jpos.gl.AccountLockTest > executionError FAILED
    org.hibernate.HibernateException: Detected concurrent management of connection resources. This might indicate a multi-threaded use of Hibernate in combination with managed resources, which is not supported.
        at org.hibernate.resource.jdbc.internal.LogicalConnectionManagedImpl.releaseConnection(LogicalConnectionManagedImpl.java:221)
        at org.hibernate.resource.jdbc.internal.LogicalConnectionManagedImpl.close(LogicalConnectionManagedImpl.java:258)
        at org.hibernate.engine.jdbc.internal.JdbcCoordinatorImpl.close(JdbcCoordinatorImpl.java:175)
        at org.hibernate.internal.AbstractSharedSessionContract.close(AbstractSharedSessionContract.java:342)
        at org.hibernate.internal.SessionImpl.closeWithoutOpenChecks(SessionImpl.java:385)
        at org.hibernate.internal.SessionImpl.close(SessionImpl.java:361)
        at org.jpos.ee.DB.close(DB.java:330)
        at org.jpos.gl.GLSession.close(GLSession.java:1520)
        at org.jpos.gl.TestBase.tearDownBase(TestBase.java:100)

@ar ar merged commit c9144d7 into jpos:master Dec 10, 2020
@jameshilliard jameshilliard deleted the actions branch December 10, 2020 16:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants