Page MenuHomePhabricator

Provide more information to users during `git push` while waiting for write locks
ClosedPublic

Authored by epriestley on Mar 22 2018, 2:11 AM.
Tags
None
Referenced Files
F13231906: D19247.diff
Tue, May 21, 12:52 AM
F13210811: D19247.diff
Fri, May 17, 5:14 AM
F13199904: D19247.id46080.diff
Mon, May 13, 8:02 PM
F13194734: D19247.diff
Sun, May 12, 9:50 PM
F13190835: D19247.id.diff
Sat, May 11, 2:19 PM
F13190287: D19247.id46086.diff
Sat, May 11, 9:16 AM
F13182926: D19247.id46081.diff
Fri, May 10, 5:12 AM
F13177500: D19247.diff
Wed, May 8, 7:45 PM
Subscribers

Details

Summary

Ref T13109. Make it slightly more clear what the scope of the write and read locks are, and slightly more clear that we're actively acquiring locks, not just sitting around waiting.

While waiting on another writer, show who we're waiting on so you can walk over to their desk and glare at them.

Test Plan

Added sleep(15) after willWrite(). Pushed in two windows. Saw new, more informative messages. In the second window, saw the new guidance:

  1. Waiting for hector to finish writing (on device "repo1.local.phacility.net" for 11s)...

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

asherkin added inline comments.
src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php
789

Missing return?

  • Add a missing return; for the case when the locking writer hasn't written their durable lock row yet.
This revision is now accepted and ready to land.Mar 22 2018, 8:15 PM
This revision was automatically updated to reflect the committed changes.