Page MenuHomePhabricator

Document all the hypothetical URI features we plan to support soon
ClosedPublic

Authored by epriestley on Apr 28 2016, 8:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 25, 7:40 PM
Unknown Object (File)
Wed, Mar 20, 4:57 PM
Unknown Object (File)
Wed, Mar 20, 4:07 PM
Unknown Object (File)
Wed, Mar 20, 4:00 PM
Unknown Object (File)
Wed, Mar 20, 2:48 PM
Unknown Object (File)
Wed, Mar 20, 2:34 PM
Unknown Object (File)
Wed, Mar 20, 2:33 PM
Unknown Object (File)
Wed, Mar 20, 1:42 PM
Subscribers

Diff Detail

Repository
rP Phabricator
Branch
repo13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11972
Build 15058: Run Core Tests
Build 15057: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Document all the hypothetical URI features we plan to support soon.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 28 2016, 8:40 PM
scode added inline comments.
src/docs/user/userguide/diffusion_uris.diviner
88

FWIW, a pitfall we ran into was that during the initial import, unless you too steps to avoid it, the initial import generates email. We ended up disabling autoclose and notify/push during the initial import to avoid this.

This may have been PEBKAC or a problem specific to us; I just thought I'd mention it in case it affects normal users and may be worth warning about here.

eadler added inline comments.
src/docs/user/userguide/diffusion_uris.diviner
88

this is the documented reason for those knobs existing. T9554 is related.

I'm guessing you probably hit T9554; I'll either fix that or caution about this before removing the scary header from this document. I think it will automatically fix itself when the distinction between "hosted" and "imported" repositories partially dissolves Fairly Soon Now, but I'll make a note for myself to double check this.

epriestley edited edge metadata.
  • "specific specific", comma here and there.
This revision was automatically updated to reflect the committed changes.