Uploaded image for project: 'JGroups'
  1. JGroups
  2. JGRP-1887

Remove the synchronization on JChannel.class in JChannel.init

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • Major
    • 3.5.2, 3.6
    • 3.5.1
    • None

    Description

      JChannel.init(ProtocolCtackConfigurator) has a synchronized(Channel.class) block that doesn't seem necessary, as JChannel instances do not normally share resources.

      This can cause serious delays in the Infinispan test suite, because we run many tests in parallel, each creating its own cluster. In some tests we also have a timeout for the test to create the JChannel (among other things), and these delays can lead to random failures (e.g. ISPN-4802).

      Attachments

        Issue Links

          is related to

          Bug - A problem that impairs or prevents the functions of the product. JGRP-964 shared_transport concurrency failure - starting NotificationBusses at the same time causes "timer cannot be retrieved from protocol stack"

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Resolved
          relates to

          Bug - A problem that impairs or prevents the functions of the product. ISPN-4802 HotRodConcurrentStartTest.testConcurrentStartup random failures

          • Blocker - To be worked above all other priorities. This term is chosen when the severity of the issue is very high, has no workaround, or the effort for the change is comparatively low. Issues that may be very publicly visible and could generate significant media attention may also drive a higher priority.
          • Closed

          Activity

            People

              rhn-engineering-bban Bela Ban
              dberinde@redhat.com Dan Berindei (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: