Skip to main content


Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Eclipse Projects » scout » Memory leak in table fields
Memory leak in table fields [message #1778506] Fri, 15 December 2017 10:35 Go to next message
Simon Meer is currently offline Simon MeerFriend
Messages: 7
Registered: February 2017
Junior Member
It seems like there may be a memory leak in tables that update often.

In Table.js, there is an array _updatedRows which stores all rows that have been updated during the lifetime of the table field:

scout.TableField.prototype._onTableChanged = function(event) {
  [...]
  if (event.type === 'rowsUpdated') {
    scout.arrays.pushAll(this._updatedRows, event.rows);
    return;
  }
  [...]
};


However, since it's an array, it will keep growing for every row that has been touched in any way. Our table can have up to 200 rows that update every second, resulting in a... quite big heap in a short time.

Suggestion:
Could _updatedRows perhaps be removed in favor of simply setting touched to true?

[Updated on: Fri, 15 December 2017 13:48]

Report message to a moderator

Re: Memory leak in table fields [message #1778594 is a reply to message #1778506] Mon, 18 December 2017 11:51 Go to previous message
Claudio Guglielmo is currently offline Claudio GuglielmoFriend
Messages: 169
Registered: March 2010
Senior Member
Hi Simon

Thank you for your feedback! This is a memory leak indeed. It will be fixed with the next builds for the 6.1, 7.0 and 7.1 branches.

If you are interested in the solution, check out the commit: https://git.eclipse.org/c/scout/org.eclipse.scout.rt.git/commit/?h=releases/6.1.x&id=ecd78e68082f8f7ccddc4c424a647a3ac6c5e87a

Best regards
Claudio
Previous Topic:Checkbox of checkable table disappearing
Next Topic:Custom login on scout
Goto Forum:
  


Current Time: Mon Sep 24 15:19:01 GMT 2018

Powered by FUDForum. Page generated in 0.05442 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top