Project

General

Profile

Actions

Bug #21444

open

getting the list of directives that would apply to a pending node is painfully slow

Added by Nicolas CHARLES over 1 year ago. Updated about 1 month ago.

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

Description

the processDynGroup method take about 50seconds to process 460 group
real reason is that we pass the nodeId to the method, and it calls "check" for each of the group. And this method calls nodeInfoService.getPendingNodeInfos() to get all nodes infos, and this is slow because it does ldap query.
We should change the signature of this method to use limitToNodeInfos:Option[Seq[NodeInfo]] and prevent the 460 ldap query

Actions #1

Updated by Alexis Mousset over 1 year ago

  • Target version changed from 6.2.16 to 6.2.17
Actions #2

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.2.17 to 997
Actions #3

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 997 to 6.2.18
Actions #4

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.2.18 to 6.2.19
Actions #5

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.2.19 to 6.2.20
Actions #6

Updated by Nicolas CHARLES over 1 year ago

  • Regression set to No

this is still painfully slow

Actions #7

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.2.20 to old 6.2 issues to relocate
Actions #8

Updated by Nicolas CHARLES 11 months ago

  • Target version changed from old 6.2 issues to relocate to 7.2.7
Actions #9

Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 7.2.7 to 7.2.8
Actions #10

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 7.2.8 to 7.2.9
Actions #11

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 7.2.9 to 7.2.10
Actions #12

Updated by Alexis Mousset 8 months ago

  • Target version changed from 7.2.10 to 7.2.11
Actions #13

Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 7.2.11 to 1046
Actions #14

Updated by Alexis Mousset 6 months ago

  • Target version changed from 1046 to 7.3.8
Actions #15

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.8 to 7.3.9
Actions #16

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 7.3.9 to 7.3.10
Actions #17

Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 7.3.10 to 7.3.11
Actions #18

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 7.3.11 to 7.3.12
Actions #19

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 7.3.12 to 7.3.13
Actions #20

Updated by Vincent MEMBRÉ about 1 month ago

  • Target version changed from 7.3.13 to 7.3.14
Actions

Also available in: Atom PDF