Bug 44879 – Memory leak in Grids

Bug 44879 - Memory leak in Grids


Summary:	Memory leak in Grids

Status:	NEW

Product:	UCS
Classification:	Unclassified
Component:	UMC (Generic)
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 minor (vote)
Target Milestone:	---
Assigned To:	UMC maintainers
QA Contact:	UMC maintainers

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2017-06-28 11:24 CEST by Johannes Keiser
Modified:	2019-03-08 14:11 CET (History)
CC List:	1 user (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?:	4: Will affect most installed domains
How will those affected feel about the bug?:	1: Nuisance – not a big deal but noticeable
User Pain:	0.023
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	Cleanup
Max CVSS v3 score:

Flags:	keiser: Patch_Available+

Attachments
patch (3.19 KB, patch) 2017-06-28 11:26 CEST, Johannes Keiser	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Johannes Keiser

2017-06-28 11:24:44 CEST

Every umc/widgets/Grid leaks one dgrid widget.

Comment 1 Johannes Keiser

2017-06-28 11:26:00 CEST

Created attachment 8976 [details]
patch

The patch removes dijit.Destoryable from the inheritance chain of the internal
dgrid widget since its functionalty is not used and it just eats the destroy() call.

This patch also tracks all created dojo/on and dojo/aspect handles with own()
to remove them when umc/widgets/Grid is destroyed.

Comment 2 Ole Schwiegert

2019-03-08 14:11:19 CET

During the product test I got a significant increase of widgets in the registry from 95 to 451 after roughly 15min of use. So we might want to have another look at this.