Ticket #2105 (new defect)

Opened 10 years ago

Last modified 9 years ago

BaseDirectory's isVisible does useless entry lookup

Reported by: gracinet Owned by: madarche
Priority: P2 Milestone: CPS 3.5.7
Component: CPSDirectory Version: TRUNK
Severity: normal Keywords: PERFORMANCE


A bit more of benchmarking in anonymous mode. isVisible() is used by listVisibleDirectories. It seems to make use of the general entry local roles, and maybe even try and fetch an entry.

Measured the performance impact very roughly on the same setup at for #2104. Returning False directly makes the requests per second go from 10.4 to 11.5 with ab -n 100

A bit more tricky and dangerous to fix properly than #2104, though

Change History

comment:1 Changed 9 years ago by gracinet

  • Milestone changed from CPS 3.5.0 to CPS 3.5.2

We can wait

Note: See TracTickets for help on using tickets.