Uploaded image for project: 'Infinispan'
  1. Infinispan
  2. ISPN-10070

DefaultCacheManager should stop components after start failure

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 9.4.16.Final, 10.0.0.Final
    • 9.4.10.Final, 10.0.0.Final
    • Core
    • None

      Currently it is impossible to release all the resources allocated during startup if the DefaultCacheManager instance was created with start=true. The user has to do something like this:

      DefaultCacheManager manager = new DefaultCacheManager(..., false);
      try {
         manager.start();
      } catch (Throwable t) {
         manager.stop();
         throw t;
      }
      

      Both the constructor and the public start() method should clean up the started components after a startup failure, so that the user doesn't have to call stop() explicitly.

      Our tests do not currently call stop() explicitly, so they leak threads and sockets when a manager fails to start (e.g. because something went wrong with the CONFIG cache).

              dberinde@redhat.com Dan Berindei (Inactive)
              dberinde@redhat.com Dan Berindei (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: