Bug 52270 - Replace xml.etree.cElementTree with lxml.etree
Replace xml.etree.cElementTree with lxml.etree
Status: NEW
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: UMC maintainers
UMC maintainers
:
Depends on: 52253
Blocks:
  Show dependency treegraph
 
Reported: 2020-10-27 13:36 CET by Florian Best
Modified: 2020-10-28 23:52 CET (History)
3 users (show)

See Also:
What kind of report is it?: Development Internal
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): UCS Performance
Max CVSS v3 score:


Attachments
patch from Arvid (6.87 KB, patch)
2020-10-28 23:52 CET, Florian Best
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2020-10-27 13:36:10 CET
(In reply to Arvid Requate from comment #1)
> lxml.etree might be even better but has a different API.

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

UMC/SAML-Login performance tests with 82_saml/90_concurrent_saml_login_multiprocessing  showed that UMC login would perform better with xml.etree.cElementTree instread of xml.etree.ElementTree. With the C implementation UMC Login performed 29% better.

* 90_concurrent_saml_login_multiprocessing against normal UMC:
## 32 cores
Mean time per portal login 0.203379382133 
Number of logins per second: 4.9169192546

* 90_concurrent_saml_login_multiprocessing against patched UMC:
UMC mit xml.etree.cElementTree statt xml.etree.ElementTree
## 32 cores
Mean time per portal login 0.157018792152 
Number of logins per second: 6.36866445278


The test case was 10 times 50 parallel (multiprocessing) logins.
Comment 1 Florian Best univentionstaff 2020-10-28 23:52:22 CET
Created attachment 10540 [details]
patch from Arvid