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

Count queries issued by Cockpit can be expensive

    • Icon: Bug Report Bug Report
    • Resolution: Won't Fix
    • Icon: L3 - Default L3 - Default
    • None
    • None
    • cockpit
    • None

      Scenario:

      • Cockpit makes count queries for two purposes: 1) Displaying the exact number of search results in a list view; 2) Displaying the exact number of result pages
      • For large datasets and unselective searches, these queries can run for a long time (or event time out) and put significant load on the database server

      Proposed improvement:

      • No longer display the number of pages; this has very little value for the user and is not worth the expensive queries (e.g. Google search also doesn't give the exact number of pages); some people even consider this an anti-pattern when searching with relational databases (e.g. https://dzone.com/articles/why-most-programmers-get-pagination-wrong)
      • Page navigation will then allow you to navigate back and forth between pages, but not to the last page or a specific page anymore.
      • For the count that is displayed next to the search result: If runtime data is displayed, show the count immediately (this should cover tasklist). If history data is displayed, show the count only on demand (e.g. have a button to resolve the count).
      • Batch creation view: Always show the count when the user has decided to create the batch. In this case, show it in a manner that the user is unlikely to miss (motivation: showing the size of the batch is a tool for our users to notice mistakes before making them)

        This is the controller panel for Smart Panels app

            [CAM-9642] Count queries issued by Cockpit can be expensive

            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
              thorben.lindhauer Thorben Lindhauer
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: