Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1169 I can upgrade Optimize
  3. OPT-1269

Use json path to define adjustments within the upgrade steps

    Details

    • Type: Sub-task
    • Status: Done
    • Priority: L3 - Default
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.0-alpha1, 2.2.0
    • Component/s: backend
    • Labels:
      None

      Description

      AT:

      • all upgrade steps use json path, if possible, to define which field they want to modify instead of defining a complex mapping
      • all step operations can still be performed

      Context:
      Define a mapping provides the developer with a lot of power, but makes it really hard to write/understand an upgrade step. Also we should avoid to write json in java as much as possible, since it's hard to read and a pain in the a**.

        Activity

        Hide
        sebastian.stamm Sebastian Stamm added a comment -

        Looks good

        We could think about passing a configuration object to the constructors of the AddFieldStep or RenameFieldStep classes. Right now they take five parameters each, which might cause confusion what is needed and in what order. But since this is a matter of taste and the other code looks good, I will close this ticket.

        Show
        sebastian.stamm Sebastian Stamm added a comment - Looks good We could think about passing a configuration object to the constructors of the AddFieldStep or RenameFieldStep classes. Right now they take five parameters each, which might cause confusion what is needed and in what order. But since this is a matter of taste and the other code looks good, I will close this ticket.

          People

          • Assignee:
            Unassigned
            Reporter:
            johannes.heinemann Johannes Heinemann
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: