Project

General

Profile

Actions
Copy link

Bug #16283

closed

we are always computing compliance and byNodes on RuleStatusReport, even when it's not necessay

Added by Nicolas CHARLES over 4 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
N/A
Assignee:
François ARMAND
Category:
Performance and scalability
Target version:
5.0.18
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

on tests in PR changing the val to lazy val makes it run from 3700ms to 2700ms

in most case, we should need compliance, but not ByNodes (used only in display)

Actions
Copy link
 #1

Updated by Nicolas CHARLES over 4 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions
Copy link
 #2

Updated by Nicolas CHARLES over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/2637
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions
Copy link
 #4

Updated by Alexis Mousset about 4 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions
Copy link
 #6

Updated by Nicolas CHARLES almost 4 years ago

  • Status changed from Pending technical review to Rejected

implemented in #16661

Actions
Copy link

Also available in: Atom PDF