Actions
Architecture #2878
closedDoes lazy for injected services matter?
Status:
Rejected
Priority:
3
Assignee:
-
Category:
Architecture - Code maintenance
Target version:
Pull Request:
Effort required:
Name check:
Fix check:
Regression:
Description
When correcting #2826 (cf: commit ddb9c4efefe1b7cb4d36c2d0010dde2b92ef86de), some "lazy val" were introduced in place of the old "val".
The reason if that it seems to be faster that way.
So, we have to:
- check if there is an actual semantic difference between the two;
- if not, try to assess what is the faster in Rudder way of using them;
- if we can't assess that, decide for a policy to use, and use it everywhere.
Actions