This is part of LB doing wearing way too many hats. LB's job is to take an index out of a hat, not to init the db object. We have a factory for that already. Also completely get rid of injecting DBFactory as a parameter in config of LB and LBF, It doesn't make any sense to pass this around as a configuration option, it's a php class, so can't even be properly set in many systems (code should not be a configuration). On top of that it's making multiple ways to override configurations with non-obvious priority that can easily lead to outages. Bug: T326274 Change-Id: I1e0c38cd3b378669d0940b9f243b61cb64c193b7 |
||
|---|---|---|
| .. | ||
| DatabaseTestHelper.php | ||
| LBFactoryTest.php | ||
| LoadBalancerTest.php | ||