Project

General

Profile

Actions
Copy link

Bug #15042

closed

Rule changes SQL request and data structure use too much memory

Bug #15042: Rule changes SQL request and data structure use too much memory

Added by Nicolas CHARLES over 6 years ago. Updated over 6 years ago.

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

Description

When we update changes, we pass, to a synchronized method, a Seq[ResultRepairedReport], to use only the RuleId and the ExecutinTimeStamp, once cache is init (see NodeChangesService, method update)
On a fairly loaded system, this can cause 28960 reports to be kept in memory for nothing, in different Futures

Subtasks 1 (0 open — 1 closed)

Bug #15129: Queuing "changes" must be done in 5.1 ReleasedFrançois ARMANDActions

Updated by Nicolas CHARLES over 6 years ago Actions
Copy link
 #1

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES

Updated by Nicolas CHARLES over 6 years ago Actions
Copy link
 #2

  • 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/2247

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #3

  • Status changed from Pending technical review to Discussion
  • Assignee changed from François ARMAND to Nicolas CHARLES

Updated by François ARMAND over 6 years ago Actions
Copy link
 #4

  • Status changed from Discussion to In progress
  • Assignee changed from Nicolas CHARLES to François ARMAND

I'm taking over this issue!

Updated by François ARMAND over 6 years ago Actions
Copy link
 #5

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request changed from https://github.com/Normation/rudder/pull/2247 to https://github.com/Normation/rudder/pull/2270

Updated by François ARMAND over 6 years ago Actions
Copy link
 #6

  • Status changed from Pending technical review to In progress
  • Assignee changed from Nicolas CHARLES to François ARMAND

I'm taking over this issue!

Updated by François ARMAND over 6 years ago Actions
Copy link
 #7

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request changed from https://github.com/Normation/rudder/pull/2270 to https://github.com/Normation/rudder/pull/2281

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #8

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Nicolas CHARLES to François ARMAND

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #9

  • Assignee changed from François ARMAND to Nicolas CHARLES

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #10

  • Assignee changed from Nicolas CHARLES to François ARMAND

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #11

  • Assignee changed from François ARMAND to Nicolas CHARLES

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #12

  • Assignee changed from Nicolas CHARLES to François ARMAND

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #13

  • Assignee changed from François ARMAND to Nicolas CHARLES

Updated by François ARMAND over 6 years ago Actions
Copy link
 #14

  • Status changed from Discussion to Pending technical review

Updated by Rudder Quality Assistant over 6 years ago Actions
Copy link
 #15

  • Assignee changed from Nicolas CHARLES to François ARMAND

Updated by Nicolas CHARLES over 6 years ago Actions
Copy link
 #16

  • Status changed from Pending technical review to Pending release

Updated by Alexis Mousset over 6 years ago Actions
Copy link
 #17

  • Name check set to To do
  • Fix check set to To do

Updated by Alexis Mousset over 6 years ago Actions
Copy link
 #18

  • Name check changed from To do to Needs change

Updated by François ARMAND over 6 years ago Actions
Copy link
 #19

  • Subject changed from Updating changes/repairs (probably) uses too much memory to Changes request and data structure use too much memory

Updated by François ARMAND over 6 years ago Actions
Copy link
 #20

  • Fix check changed from To do to Error - Blocking

Updated by François ARMAND over 6 years ago Actions
Copy link
 #21

  • Subject changed from Changes request and data structure use too much memory to Rule changes SQL request and data structure use too much memory

Updated by François ARMAND over 6 years ago Actions
Copy link
 #22

  • Name check changed from Needs change to To do

Updated by François ARMAND over 6 years ago Actions
Copy link
 #23

  • Fix check changed from Error - Blocking to Error - Fixed

Updated by Vincent MEMBRÉ over 6 years ago Actions
Copy link
 #24

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 5.0.12 which was released today.

Actions
Copy link

Also available in: PDF Atom