Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-1363

decrease ChatTranscriptPlugin loading time and Ram

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Critical
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Unknown
    • Component/s: None
    • Labels:
      None

      Description

      need to speedup history loading
      and remove the 5000message limitation

        Attachments

          Activity

          Hide
          mikemcmahon Mike McMahon added a comment -

          Pretty messy, not what i would put in the source.

          Basically we can speed up the portion of loading/dumping into the JEditorPane to under a second (on certain systems) but it's the drawing on the JEditorPane that's killing us. You can (in the code) change the window.setText() / window.setPage() (line 473/474 in ChatTranscriptPlugin) to see how it affects the performances (on the console output).

          It becomes painfully apparent where our slowness is when you use the setPage() method!

          Show
          mikemcmahon Mike McMahon added a comment - Pretty messy, not what i would put in the source. Basically we can speed up the portion of loading/dumping into the JEditorPane to under a second (on certain systems) but it's the drawing on the JEditorPane that's killing us. You can (in the code) change the window.setText() / window.setPage() (line 473/474 in ChatTranscriptPlugin) to see how it affects the performances (on the console output). It becomes painfully apparent where our slowness is when you use the setPage() method!
          Hide
          wroot wroot added a comment -

          I didn't notice much improvement on huge history loading. It takes same amount of time. Maybe a scrollbar appears faster, but it is still not possible to use it for some time.

          Show
          wroot wroot added a comment - I didn't notice much improvement on huge history loading. It takes same amount of time. Maybe a scrollbar appears faster, but it is still not possible to use it for some time.
          Hide
          mikemcmahon Mike McMahon added a comment -

          Yeah it's the drawing that's killing us unfortunately. the setPage loads faster but once it calls to draw it takes roughly the same amount of time.

          Show
          mikemcmahon Mike McMahon added a comment - Yeah it's the drawing that's killing us unfortunately. the setPage loads faster but once it calls to draw it takes roughly the same amount of time.
          Hide
          mikemcmahon Mike McMahon added a comment -

          please test the newest patch, i switch to using invokeLater for loading the text vs. nothing at all.

          This seems to cut down on the loading by about half on my system. Though i would love to get some more input from others.

          Show
          mikemcmahon Mike McMahon added a comment - please test the newest patch, i switch to using invokeLater for loading the text vs. nothing at all. This seems to cut down on the loading by about half on my system. Though i would love to get some more input from others.
          Hide
          wroot wroot added a comment -

          Looks like ChatTranscriptPlugin.java has evolved much since these patches (using TextArea), so can't apply and test them.

          Show
          wroot wroot added a comment - Looks like ChatTranscriptPlugin.java has evolved much since these patches (using TextArea), so can't apply and test them.

            People

            • Assignee:
              Unassigned
              Reporter:
              wolf.posdorfer Wolf P.
            • Votes:
              2 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: