Page MenuHomePhabricator

Record lock timing information on PushEvents
ClosedPublic

Authored by epriestley on Mar 22 2018, 8:09 PM.
Tags
None
Referenced Files
F13129781: D19250.diff
Wed, May 1, 2:48 AM
Unknown Object (File)
Mon, Apr 29, 6:01 PM
Unknown Object (File)
Mon, Apr 29, 4:01 PM
Unknown Object (File)
Mon, Apr 29, 2:17 AM
Unknown Object (File)
Wed, Apr 24, 11:12 PM
Unknown Object (File)
Sun, Apr 21, 2:29 PM
Unknown Object (File)
Thu, Apr 11, 7:49 AM
Unknown Object (File)
Sun, Apr 7, 5:43 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.