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

        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: