[SERVER-71476] Disparity between SemaphoreTicketHolder and PriorityTicketHolder queueing in benchmarks Created: 18/Nov/22  Updated: 27/Oct/23  Resolved: 02/Feb/23

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Major - P3
Reporter: Haley Connelly Assignee: Haley Connelly
Resolution: Works as Designed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File image-2023-02-02-14-43-34-370.png    
Issue Links:
Related
related to SERVER-71213 High variance in PriorityTicketHolder... Closed
related to SERVER-71829 Add latency distribution of operation... Closed
Sprint: Execution Team 2022-12-12, Execution Team 2022-11-28, Execution Team 2023-01-23, Execution Team 2023-02-06
Participants:

 Comments   
Comment by Haley Connelly [ 02/Feb/23 ]

The benchruns serve as an upper bound on the performance regressions when the PriorityTicketHolder is enabled. As each ticket is immediately acquired and then released, which is unlikely in a real system since a ticket usually is held for some amount of work.

Comment by Haley Connelly [ 02/Feb/23 ]


With the new implementation of the PriorityTicketHolder SERVER-72072, the issue of "skipping the line" in the PriorityTicketHolder has been addressed. Above are the average performance numbers for each type of TicketHolder when ticketholder_bm.cpp is run 100 times. 

Comment by Haley Connelly [ 19/Jan/23 ]

With the completion of SERVER-72072, we are ready to revisit the benchmark performance of the PriorityTicketHolder

Comment by Haley Connelly [ 05/Dec/22 ]

While performance is under investigation, we are working on SERVER-71829 to add latency distribution on the benchmarks so it is clear if the latency distribution is off in the future.

Comment by Haley Connelly [ 18/Nov/22 ]

A full description can be found in the comments of SERVER-71213

Generated at Thu Feb 08 06:19:07 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.