Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-13509

Consolidate duplicated code for definition removal from deployment cache

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Won't Fix
    • Icon: L3 - Default L3 - Default
    • None
    • 7.15.0
    • engine
    • None

      What is the problem?

      The DeploymentCache.removeDeployment method (https://github.com/camunda/camunda-bpm-platform/blob/master/engine/src/main/java/org/camunda/bpm/engine/impl/persistence/deploy/cache/DeploymentCache.java#L310-L319) performs the same functionality as part of the DeploymentManager.deleteDeployment method (https://github.com/camunda/camunda-bpm-platform/blob/master/engine/src/main/java/org/camunda/bpm/engine/impl/persistence/entity/DeploymentManager.java#L126-L129).

      AC:
      Reduce code duplication for the following features:

      • Remove definitions from CmmnModelInstanceCache by deployment id;
      • Remove definitions from DmnModelInstanceCache by deployment id;
      • Remove DecisionRequirementsDefinitions for a deployment;

      Hints (optional):
      The duplication was reported in CAM-13205 and PR 1365.

        This is the controller panel for Smart Panels app

            [CAM-13509] Consolidate duplicated code for definition removal from deployment cache

            Tobias Metzke-Bernstein added a comment -

            nikola.koevski,

            there is a potential reduction of future errors by tackling this. However, due to other commitments and priorities we will not add this to the roadmap for now.

            Best,
            Tobias

            Tobias Metzke-Bernstein added a comment - nikola.koevski , there is a potential reduction of future errors by tackling this. However, due to other commitments and priorities we will not add this to the roadmap for now. Best, Tobias

            Thorben Lindhauer added a comment -

            We are closing this ticket as part of our backlog grooming. Reasons:

            • We did not receive sufficient evidence that this ticket is important

            Thorben Lindhauer added a comment - We are closing this ticket as part of our backlog grooming. Reasons: We did not receive sufficient evidence that this ticket is important

              Unassigned Unassigned
              nikola.koevski Nikola Koevski
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: