Project

General

Profile

Actions

Bug #20674

closed

when saving processes in inventories, don't fetch them to replace them

Added by Nicolas CHARLES almost 3 years ago. Updated about 1 year ago.

Status:
Rejected
Priority:
N/A
Category:
Performance and scalability
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
No

Description

we could simply replace the content with the new entries


Related issues 1 (0 open1 closed)

Related to Rudder - Architecture #23727: Group all node related access into one NodeFactRepositoryReleasedVincent MEMBRÉActions
Actions #1

Updated by Nicolas CHARLES almost 3 years ago

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

Updated by Nicolas CHARLES almost 3 years ago

commit here, that doesn't work (it empties the processes)
https://github.com/ncharles/rudder/commit/b1e9a07487025b8d8c3772d6261aa1ca8da6391c

Actions #4

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.19 to 6.1.20
Actions #5

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.20 to 6.1.21
Actions #6

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate
Actions #7

Updated by Alexis Mousset almost 2 years ago

  • Status changed from In progress to New
Actions #8

Updated by François ARMAND about 1 year ago

  • Related to Architecture #23727: Group all node related access into one NodeFactRepository added
Actions #9

Updated by François ARMAND about 1 year ago

  • Status changed from New to Rejected
  • Regression set to No

This will be changed by #23727 and when perf are too impacted, we can now avoid getting process completely ( #22359 ).
I'm closing that one, and performance will be reevaluated based on #23727 arch.

Actions

Also available in: Atom PDF