Uploaded image for project: 'RAML Module Builder'
  1. RAML Module Builder
  2. RMB-840

IllegalStateException in handleClosed (ConnectionPool)

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • P2
    • Resolution: Done
    • 33.0.0
    • 33.0.0
    • None
    • CP: sprint 113, CP: sprint 114
    • 2
    • Core: Platform

    Description

      2021-05-08T12:21:17,334 INFO  OkapiClient          909005/circulation RES 200 29044us okapiClient http://10.0.2.15:9148/circulation/scheduled-anonymize-processing
      2021-05-08T12:21:17,334 INFO  TenantManager        timer call succeeded to module mod-circulation-21.0.0-SNAPSHOT.831 for tenant diku
      2021-05-08T12:21:51,173 INFO  ProcessModuleHandle  mod-permissions-5.14.0 [Sat May 08 12:21:51 UTC 2021] io.vertx.core.impl.ContextImpl SEVERE Unhandled exception 
      2021-05-08T12:21:51,173 INFO  ProcessModuleHandle  mod-permissions-5.14.0 java.lang.IllegalStateException
      2021-05-08T12:21:51,173 INFO  ProcessModuleHandle  mod-permissions-5.14.0       at io.vertx.sqlclient.impl.pool.ConnectionPool$PooledConnection.handleClosed(ConnectionPool.java:228)
      2021-05-08T12:21:51,173 INFO  ProcessModuleHandle  mod-permissions-5.14.0       at io.vertx.sqlclient.impl.SocketConnectionBase.handleClose(SocketConnectionBase.java:328)
      2021-05-08T12:21:51,173 INFO  ProcessModuleHandle  mod-permissions-5.14.0       at io.vertx.sqlclient.impl.SocketConnectionBase.handleClosed(SocketConnectionBase.java:300)
      

      Line 228 in ConectionPool.java: https://github.com/folio-org/vertx-sql-client/blob/v4.0.0-FOLIO/vertx-sql-client/src/main/java/io/vertx/sqlclient/impl/pool/ConnectionPool.java#L228

      After a long session on the Vagrant box with git bisect we end up with:

      commit 5bc3b6d221628721639c019b781933eb0ac6bb85
      Author: Julian Ladisch <julianladisch@users.noreply.github.com>
      Date:   Tue Mar 16 18:32:52 2021 +0100
      
          RMB-784: Reenable DB_CONNECTIONRELEASEDELAY
      
       .../src/main/java/org/folio/rest/persist/PostgresClient.java        | 6 ++----
       1 file changed, 2 insertions(+), 4 deletions(-)
      

      Actually not surprising given where the exception is thrown.

      Given the seriousness of this error, I suggest for this issue we disable this for now.. And make a follow up for a proper solution.

      33.0.0 pre1 does include this commit. But 33.0.0 pre2 does.

      TestRail: Results

        Attachments

          Issue Links

            Activity

              People

                adam Adam Dickmeiss (Inactive)
                adam Adam Dickmeiss (Inactive)
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases