Uploaded image for project: 'ZK Spreadsheet'
  1. ZK Spreadsheet
  2. ZSS-1181

notify a loaded range is slower than notify an affected range

    Details

    • Type: Bug
    • Status: Reopened
    • Priority: Later
    • Resolution: Fixed
    • Affects Version/s: 3.8.2
    • Fix Version/s: 3.8.3
    • Component/s: None
    • Security Level: Jimmy
    • Labels:
      None
    • gh.sprint.customfield.default.name:
      Sprint 3, Sprint 4

      Description

      steps to reproduce

      1. run the attached project (version 3.8.3.FL.20160106-Eval)
      2. test under Firefox 42
      3. in merge(), call notifyAffectedRange(event);
      4. click headers to insert columns upon the order below
        9103
        9131
        9119
        9160
        9150
        9170
        9105
        9110
      5. in merge() , change to call notifyLoadedRange();, perform the previous step

      result

      • see attached report
      • notifyLoadedRange(); is slower than another approach.
        Execution time: measured from clicking the header cell to seeing a rendering complete.
      • But the number of cells in this range is smaller (225120) than another (325242). The loaded range doesn't reduce during scrolling, it starts from A1:DE201 to A1:FO201 during testing
      • notifyLoadedRange(); have average bigger response size than another.

      expected result

      • notifyLoadedRange(); is faster than notifyAffectedRange(event);
      • The time taken by notifyLoadedRange(); doesn't increase by more and more column insertions.
      1. zss1181TestReport.xlsx
        38 kB
        hawk
      1. cellCountNotifyChange.png
        37 kB
      2. clickHeader.png
        50 kB
      3. comparison.png
        34 kB
      4. executionTimeComparison.png
        25 kB
      5. responseSizeComparison.png
        20 kB

        Issue Links

          Activity

          Hide
          henrichen henrichen added a comment -

          Fixed.

          Show
          henrichen henrichen added a comment - Fixed.

            People

            • Assignee:
              henrichen henrichen
              Reporter:
              hawk hawk
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Agile