Project

General

Profile

Actions
Copy link

Bug #21444

open

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

Added by Nicolas CHARLES almost 2 years ago. Updated 15 days ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Performance and scalability
Target version:
7.3.17
Pull Request:
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
Copy link
 #1

Updated by Alexis Mousset almost 2 years ago

  • Target version changed from 6.2.16 to 6.2.17
Actions
Copy link
 #2

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 6.2.17 to 997
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 997 to 6.2.18
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 6.2.18 to 6.2.19
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 6.2.19 to 6.2.20
Actions
Copy link
 #6

Updated by Nicolas CHARLES almost 2 years ago

  • Regression set to No

this is still painfully slow

Actions
Copy link
 #7

Updated by Vincent MEMBRÉ almost 2 years ago

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

Updated by Nicolas CHARLES about 1 year ago

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

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.2.7 to 7.2.8
Actions
Copy link
 #10

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.2.8 to 7.2.9
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 7.2.9 to 7.2.10
Actions
Copy link
 #12

Updated by Alexis Mousset 11 months ago

  • Target version changed from 7.2.10 to 7.2.11
Actions
Copy link
 #13

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 7.2.11 to 1046
Actions
Copy link
 #14

Updated by Alexis Mousset 9 months ago

  • Target version changed from 1046 to 7.3.8
Actions
Copy link
 #15

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 7.3.8 to 7.3.9
Actions
Copy link
 #16

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 7.3.9 to 7.3.10
Actions
Copy link
 #17

Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 7.3.10 to 7.3.11
Actions
Copy link
 #18

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.11 to 7.3.12
Actions
Copy link
 #19

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 7.3.12 to 7.3.13
Actions
Copy link
 #20

Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 7.3.13 to 7.3.14
Actions
Copy link
 #21

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 7.3.14 to 7.3.15
Actions
Copy link
 #22

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 7.3.15 to 7.3.16
Actions
Copy link
 #23

Updated by Vincent MEMBRÉ 15 days ago

  • Target version changed from 7.3.16 to 7.3.17
Actions
Copy link

Also available in: Atom PDF