Uploaded image for project: 'Geode'
  1. Geode
  2. GEODE-7531

PoolManagerImpl.unregister(:Pool) naively assumes all Pool object instances are PoolImpls

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.10.0
    • Fix Version/s: 1.11.0, 1.12.0
    • Component/s: client/server
    • Labels:
      None
    • Environment:
      Apache Geode based applications on the JVM.

      Description

      A recent change to the o.a.g.internal.cache.PoolManagerImpl class expects all o.a.g.cache.client.Pools registered with the o.a.g.cache.client.PoolManager to be o.a.g.cache.client.internal.PoolImp objects.

      This is certainly not going to be the case for Unit Tests that properly "mock" 1 or more Pool instances and additionally needs to register the mock Pool instances with the PoolManager. While the later may not be as common for application code, it is more certainly common, and in some cases necessary, for framework or tooling code.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ukohlmeyer Udo Kohlmeyer
                Reporter:
                jblum John Blum
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m