Bug 29647 - Geschwindigkeits- und Speicheroptimierungen in UMC nötig (2)
Geschwindigkeits- und Speicheroptimierungen in UMC nötig (2)
Status: RESOLVED WORKSFORME
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.1
Other Linux
: P3 normal (vote)
: UCS 3.x
Assigned To: UMC maintainers
:
Depends on: 27432
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-07 14:57 CET by Dirk Wiesenthal
Modified: 2016-10-07 18:22 CEST (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Browser compatibility
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Wiesenthal univentionstaff 2012-12-07 14:57:03 CET
Die Optimierungsarbeiten sind noch nicht abgeschlossen. Ich habe einmal jedes Modul geöffnet wild herumgeklickt und wieder geschlossen. Ergebnis: Einstmals 185 Widgets, danach 300.

Leider kann ich nicht mehr genau sagen, welches Widget wann nicht aufgeräumt wurde (nicht systematisch genug getestet). Aber folgender Verdacht: AD-Connektor-Konfigurations-Dialog, UDM-Report-Dialog (extrahiert aus Button-Texten etc). Überhaupt sind es vor allem die Dialoge, die nicht richtig aufgeräumt werden. Grund ist, dass viele so eine komische _cleanup-Funktion definieren. Die wird aber nur bei "Ok" oder "Cancel" ausgeführt. Nicht zum Beispiel beim Klick oben rechts auf das Kreuz. Sowas sollte man sowieso automatisch machen und nicht explizit in irgendwelchen Button-Callbacks. Ich empfehle so etwas (natürlich ein bisschen schicker):
  declare(OneWayDialog, [Dialog], { onHide: 'destryRecursive'})
Dann kann einem sowas nicht mehr passieren. Dialoge zu speichern lohnt sich nur ganz selten (z.B. das dialog.confirm-Singleton-Objekt).

Außerdem: Viel könnte gespart werden, wenn es nur noch einen MasterTooltip gibt, der dann dynamisch immer dorthin wandert, wo er gebraucht wird. Momentan werden jede Menge Tooltips erzeugt. Dazu sollte man aber besser eine Bibliothek oder ähnliches suchen (kann mir kaum vorstellen, dass wir die einzigen mit dem Problem sind).

Man sollte auch einmal mit dem Bündeln von udm/syntax/choices experimentieren. Das wird kompliziert, kann aber sicher einiges bewirken. Es gibt in UDM bereits ein UMCPBundle, vielleicht kann man da anknüpfen.

+++ This bug was initially created as a clone of Bug #27432 +++

Unter WinXP mit IE8 erhalte ich öfters die Meldung "Die Anfrage konnte nicht
ausgeführt werden", wenn ich die Detailseite eines Benutzers öffne. Dies kann
mit den relativ vielen parallen Ajax-Abfragen zusammenhängen. Zusätzlich sei
erwähnt, dass der Aufbau der Detailseite sehr langsam ist. Vielleicht gibt es
noch Möglichkeiten dies zu optimieren.
Comment 1 Dirk Wiesenthal univentionstaff 2012-12-07 16:14:06 CET
Man sollte auch mal nach removeChild suchen. Denn das Kind, das entfernt wird, ist nicht mehr automatisch "geownt", d.h. man muss das Widget per Hand zerstören.
Comment 2 Florian Best univentionstaff 2014-11-28 10:40:36 CET
Way more better in UCS 4.0. During product tests every module was tested. No more memory leaks only in UVMM.
Leaving this opened because of the dialog thing which is still an issue.
Comment 3 Florian Best univentionstaff 2016-10-07 18:22:45 CEST
I guess this is fine otherweise the UCS 4.2 product tests should inform about new browser side memory leaks.