Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7013

CoverageWatcher.js performance improvements

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: SUIT
    • Labels:
    • Story Points:
      16
    • Team:
      Science User Interface

      Description

      Currently, whenever new (or filtered) catalog table is received, we send a request to the server to obtain central and/or corner points for each catalog row. (Search for doFetchTable call in CatalogWatcher.js) These points are then used to compute central point and maximum radius to get the coverage image. (See CatalogWatcher:computeSize.)

      For doFetchTable request, we a get a few columns of the WHOLE table, which considerably slows the system when catalog table is huge.

      A big improvement would be replacing doFetchTable call with a server call, which takes catalog table request and calculates central point and radius on the server.

      Further optimization – if needed in future – could be adding central point and radius into table meta in prepareTableMeta of the search processor when we know it (cone search) or it is easy to calculate (polygon search).

        Attachments

          Issue Links

          is blocked by

          Bug - A problem which impairs or prevents the functions of the product. DM-11383 Coverage image is broken in API mode

          • Done
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            tatianag Tatiana Goldina created issue -
            tatianag Tatiana Goldina made changes -
            Field Original Value New Value
            Epic Link DM-6107 [ 24454 ]
            tatianag Tatiana Goldina made changes -
            Team Science User Interface [ 10303 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Summary CoverageWatcher.js improvements CoverageWatcher.js performance improvements
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Story Points 14
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Story Points 14 16
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Epic Link DM-6107 [ 24454 ] DM-8351 [ 27955 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Labels FireflyCCB
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Sprint SUIT Sprint 2017-4 [ 603 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Rank Ranked higher
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Sprint SUIT Sprint 2017-4 [ 603 ] SUIT Sprint 2017-5 [ 604 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Rank Ranked lower
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Sprint SUIT Sprint 2017-5 [ 604 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Sprint SUIT Sprint 2017-6 [ 609 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Rank Ranked higher
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Sprint SUIT Sprint 2017-6 [ 609 ] SUIT Sprint 2017-7 [ 618 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Sprint SUIT Sprint 2017-7 [ 618 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Rank Ranked higher
            ejoliet Emmanuel Joliet made changes -
            Link This issue is blocked by DM-11383 [ DM-11383 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Epic Link DM-8351 [ 27955 ] DM-8396 [ 28007 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Epic Link DM-8396 [ 28007 ] DM-4425 [ 21565 ]
            tjenness Tim Jenness made changes -
            Remote Link This issue links to "Page (Confluence)" [ 31365 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Gregory Dubois-Felsmann is this old ticket still relevant?

            Show
            tjenness Tim Jenness added a comment - Gregory Dubois-Felsmann is this old ticket still relevant?

              People

              Assignee:
              xiuqin Xiuqin Wu [X] (Inactive)
              Reporter:
              tatianag Tatiana Goldina
              Watchers:
              Tatiana Goldina, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.