HomePhorge

Remove 'initFromConduit' option from Diffusion

Description

Remove 'initFromConduit' option from Diffusion

Summary:
Ref T2783. I think this served two purposes:

  • Improving performance in cases where we "know" a repository is local.
  • Preventing loops.

It is now obsolete:

  • After D11476, refs can almost always resolve on a fast path.
  • As T2783 moves forward, we can usually no longer know when a repository is local without actually looking it up -- almost everything is allowed to run anywhere.
  • The cluster behavior in D11475 now prevents loops.

Test Plan: grep, browsed around. This didn't really do much of anything anymore.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T2783

Differential Revision: https://secure.phabricator.com/D11477

Details

Provenance
epriestleyAuthored on Jan 23 2015, 1:31 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPd98eb2c8b809: Provide a fast path for resolving repository refs
Branches
Unknown
Tags
Unknown

Event Timeline