core icon indicating copy to clipboard operation
core copied to clipboard

Log Viewer: Improve performance

Open nollymar opened this issue 3 years ago • 1 comments

Describe the bug We have identified 2 issues here:

  • [ ] When you perform a massive copy/publish content operation (copy/publish multiple pieces of content), with the popup log opened, the browser eventually crashes Screen Shot 2022-07-29 at 6 12 43 PM

  • [ ] The popup stops tailing log after leaving it open for a while. I noticed it after 3 ~ 4 hours.

Expected behavior The log viewer should display what the catalina.out contains and it shouldn't crash using or not a filter criteria.

Notes from Review meeting:

  1. The new fix will provide a brand new Server side based on events
  2. The events will still be written into the iFrame
  3. The iFRame will still serve as the log-view data source
  4. The Div that shows the lines will only hold a certain number of "pages" or groups of logs if you will
  5. Upon the arrival of new pages, a certain percentage of the old pages will be dropped
  6. The Highlight will be executed on the pages shown on the div view. Never on the whole log to speed up performance
  7. The number of logs loaded on the iframe can be limited so prior entries will be dropped depending on a param

nollymar avatar Aug 02 '22 19:08 nollymar

Here's the branch that provides the POC for partitioning the log using "page" class https://github.com/dotCMS/core/tree/improve-tail-log

fabrizzio-dotCMS avatar Aug 09 '22 20:08 fabrizzio-dotCMS

Please add your planning poker estimate with Zenhub @fabrizzio-dotCMS

nollymar avatar Sep 12 '22 20:09 nollymar

Dupe #22968

wezell avatar Oct 18 '22 18:10 wezell