Project

General

Profile

Actions
Copy link

Bug #12650

closed

Directive and Rule tree filter are very slow

Bug #12650: Directive and Rule tree filter are very slow

Added by Alexis Mousset almost 8 years ago. Updated almost 8 years ago.

Status:
Released
Priority:
N/A
Assignee:
Nicolas CHARLES
Category:
Web - UI & UX
Target version:
4.1.12
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

On a server with hundreds of directifs, the directive tree filter is very slow (~5 seconds) and blocks the application when doing a search.

Seen on Firefox 52 ESR with Rudder 4.3.1.

Subtasks 1 (0 open — 1 closed)

Bug #12690: Rule tree filter is a bit slowReleasedNicolas CHARLESActions

Updated by Alexis Mousset almost 8 years ago Actions
Copy link
 #1

  • Description updated (diff)

Updated by Vincent MEMBRÉ almost 8 years ago Actions
Copy link
 #2

  • Target version changed from 4.3.2 to 410

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #3

happens also on Edge

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #4

  • Target version changed from 410 to 4.1.13

Happens also on 4.1

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #5

Ok, it seems it is heavily manipulating the DOM, and at each character it does a change.
Searching for "test" in one JS call is pretty fast (at least, barely noticable)

$('#activeTechniquesTree').jstree('searchtag', 'test', [], false);

while simulating the typing with
$('#activeTechniquesTree').jstree('searchtag', 't', [], false);$('#activeTechniquesTree').jstree('searchtag', 'te', [], false);$('#activeTechniquesTree').jstree('searchtag', 'tes', [], false);$('#activeTechniquesTree').jstree('searchtag', 'test', [], false);

is pretty slow

as described in https://github.com/vakata/jstree/issues/1195 ( http://jsfiddle.net/no430s1f/2/ ) we should not immediately search, but rather have a small grace period for text change in the search box

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #6

  • Status changed from New to In progress
  • Assignee changed from Raphael GAUTHIER to Nicolas CHARLES

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #7

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/1945

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #8

  • Assignee changed from Vincent MEMBRÉ to Raphael GAUTHIER

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #9

  • Assignee changed from Raphael GAUTHIER to Vincent MEMBRÉ

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #10

  • Assignee changed from Vincent MEMBRÉ to Raphael GAUTHIER

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #11

  • Related to Bug #12690: Rule tree filter is a bit slow added

Updated by Rudder Quality Assistant almost 8 years ago Actions
Copy link
 #12

  • Assignee changed from Raphael GAUTHIER to Nicolas CHARLES

Updated by Nicolas CHARLES almost 8 years ago Actions
Copy link
 #13

  • Status changed from Pending technical review to Pending release

Updated by Vincent MEMBRÉ almost 8 years ago Actions
Copy link
 #14

  • Target version changed from 4.1.13 to 4.1.12

Updated by Benoît PECCATTE almost 8 years ago Actions
Copy link
 #15

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.12, 4.2.6 and 4.3.1 which were released today.

Updated by Vincent MEMBRÉ almost 8 years ago Actions
Copy link
 #16

  • Subject changed from Directive tree filter is very slow to Directive and Rule tree filter are very slow

Updated by Vincent MEMBRÉ almost 8 years ago Actions
Copy link
 #17

  • Related to deleted (Bug #12690: Rule tree filter is a bit slow)
Actions
Copy link

Also available in: PDF Atom