Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1520

Move all type name constants from configuration to ElasticsearchConstants class

    Details

    • Type: Task
    • Status: Done
    • Priority: L3 - Default
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 2.4.0-alpha1
    • Component/s: backend
    • Labels:
      None

      Description

      AT:

      • Move all Elasticsearch type and index name constants from configuration to the ElasticsearchConstants class
      • the default config file service-config.yaml also doesn't contain those entries anymore
      • the default config file in the documentation should be adjusted as well
      • Also use reference those types from the integrations tests. E.g. in the ElasticsearchIntegrationRule we have getters to retrieve the types.
      • Remove type definition from the test property files, e.g. the integration-rules.properties file

      Context:
      In the past we used to define all constants in the ConfigurationService, in order to allow the users to adjust the names, if they want to. However, that's actually something users don't want to do and also changes like that are not tested and very prone to error. Therefore, let's hard code all Elasticsearch type and index names in Optimize.

        Issue Links

          Activity

          There are no comments yet on this issue.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: