Bug 52270 - Replace xml.etree.cElementTree with lxml.etree
Summary: Replace xml.etree.cElementTree with lxml.etree
Status: RESOLVED WONTFIX
Alias: None
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
Version: UCS 4.4
Hardware: Other Linux
: P5 normal
Target Milestone: ---
Assignee: UMC maintainers
QA Contact: UMC maintainers
URL:
Keywords:
Depends on: 52253
Blocks:
  Show dependency tree / graph
 
Reported: 2020-10-27 13:36 CET by Florian Best
Modified: 2025-02-05 09:28 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
Customer ID:
Max CVSS v3 score:

Attachments
patch from Arvid (6.87 KB, patch)
2020-10-28 23:52 CET, Florian Best 		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 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
Comment 2 Jan-Luca Kiok univentionstaff 2025-02-05 09:28:32 CET 
This issue has been filed against UCS 4.4.

UCS 4.4 is out of maintenance and components may have vastly changed in later releases. Thus, this issue is now being closed.

If this issue still occurs in newer versions, please use "Clone this bug" or reopen this issue. In this case please provide information on how this issue is affecting you.