Actions
Bug #16606
closedBug #16524: we should not use .size to check if a collection is empty
same as parent ticket, but for .length
Status:
Released
Priority:
N/A
Assignee:
Category:
Performance and scalability
Target version:
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:
Description
we are using a lot of .length to check if strings are empty, but we should use isEmpty, even if perf improvement s minimal (see https://www.baeldung.com/java-string-performance)
Updated by Nicolas CHARLES almost 5 years ago
- Status changed from New to In progress
- Assignee set to Nicolas CHARLES
Updated by Nicolas CHARLES almost 5 years ago
- Description updated (diff)
- Category deleted (
Performance and scalability) - Assignee deleted (
Nicolas CHARLES) - Target version deleted (
5.0.16) - Name check deleted (
To do)
Updated by Nicolas CHARLES almost 5 years ago
- Category set to Performance and scalability
- Target version set to 5.0.16
Updated by Nicolas CHARLES almost 5 years ago
- Status changed from In progress to Pending technical review
- Assignee set to François ARMAND
- Pull Request set to https://github.com/Normation/rudder/pull/2735
Updated by Nicolas CHARLES almost 5 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder|0cf86630350151c91292e37c36b782adbfd6b42f.
Updated by François ARMAND almost 5 years ago
- Fix check changed from To do to Checked
Updated by Vincent MEMBRÉ almost 5 years ago
This bug has been fixed in Rudder 5.0.16 which was released today.
Updated by Vincent MEMBRÉ almost 5 years ago
- Related to Bug #16607: Merge error in #16606 added
Updated by Vincent MEMBRÉ almost 5 years ago
- Status changed from Pending release to Released
Actions