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

Change report duration data results should be represented as strings instead of numbers

    Details

      Description

      AT:

      • all types of duration report results have the data represented as string values instead of a number which represents millisencods since 1970
      • the upgrade script is adjusted so that the report data is changed to the new structure
      • the rest api documentation is adjusted accordingly

      Context:
      If the numbers are very large, the front-end might run into troubles interpreting the values as correct durations.

      Hint:
      A valid format could be based on the ISO 8601, e.g.

      P[YYYY]-[MM]-[DD]T[hh]:[mm]:[ss].[mmm]

        Activity

        Hide
        sebastian.stamm Sebastian Stamm added a comment -

        I am not sure how valuable that would be except for single number duration reports. As soon as we want to compare two numbers (e.g. within a bar chart or progress bar), we need them in a numeric format. A string like ISO 8601 would not help much there.

        Regarding concerns for the value range of integers in Javascript, the safe integer range is from −(2 53 −1) to 2 53 −1, which allows us to represent durations of up to 285.000 years accurately.

        Show
        sebastian.stamm Sebastian Stamm added a comment - I am not sure how valuable that would be except for single number duration reports. As soon as we want to compare two numbers (e.g. within a bar chart or progress bar), we need them in a numeric format. A string like ISO 8601 would not help much there. Regarding concerns for the value range of integers in Javascript, the safe integer range is from −(2 53 −1) to 2 53 −1, which allows us to represent durations of up to 285.000 years accurately.
        Hide
        johannes.heinemann Johannes Heinemann added a comment -

        Okay then let's not do it

        Show
        johannes.heinemann Johannes Heinemann added a comment - Okay then let's not do it
        Hide
        johannes.heinemann Johannes Heinemann added a comment -

        Closed for now. See Sebastian comment on why.

        Show
        johannes.heinemann Johannes Heinemann added a comment - Closed for now. See Sebastian comment on why.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: