Page MenuHomePhabricator

Record lock timing information on PushEvents
ClosedPublic

Authored by epriestley on Mar 22 2018, 8:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 7:49 AM
Unknown Object (File)
Sun, Apr 7, 5:43 PM
Unknown Object (File)
Sun, Mar 31, 11:25 AM
Unknown Object (File)
Sun, Mar 31, 5:45 AM
Unknown Object (File)
Sun, Mar 31, 5:45 AM
Unknown Object (File)
Sun, Mar 31, 5:45 AM
Unknown Object (File)
Feb 20 2024, 1:39 AM
Unknown Object (File)
Jan 15 2024, 5:11 PM
Subscribers
None

Details

Summary

Depends on D19249. Ref T13109. Add timing information to the PushEvent:

  • writeWait: Time spent waiting for a write lock.
  • readWait: Time spent waiting for a read lock.
  • hostWait: Roughly, total time spent on the leaf node.

The primary goal here is to see if readWait is meaningful in the wild. If it is, that motivates smarter routing, and the value of smarter routing can be demonstrated by looking for a reduction in read wait times.

Test Plan

Pushed some stuff, saw reasonable timing values in the table. Saw timing information in "Export Data".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 22 2018, 8:46 PM
This revision was automatically updated to reflect the committed changes.