#315: fix self-contradicting javadoc left by the previous commit
FixedPlacementPolicy's class javadoc still opened with "This ignores caps and reachability" after the previous commit added reachability as the fourth carve-out that is explicitly NOT ignored — caught by a shape-check survey run against this same file as part of #315's own request ("look in placement/ ... for the same shape: a caller/comment that documents an expectation ... where an implementation does not meet it"). Reworded the opening sentence: fixed still ignores caps (maxLoad) by design, but reachability is now a narrower, per-call retry exclusion, not an ignored concern.
This commit is contained in:
@@ -5,8 +5,12 @@ import java.util.List;
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Backward-compatible placement: an unqualified spawn always resolves to the configured default
|
* Backward-compatible placement: an unqualified spawn always resolves to the configured default
|
||||||
* profile, exactly as {@code CompositePeerLauncher} did before CB-518. This ignores caps and
|
* profile, exactly as {@code CompositePeerLauncher} did before CB-518. This ignores caps
|
||||||
* reachability so that a pre-existing config behaves identically after upgrade.
|
* ({@code maxLoad}) so that a pre-existing config behaves identically after upgrade — capacity
|
||||||
|
* gating for automatic placement is deliberately out of scope for {@code fixed}, exactly as it
|
||||||
|
* always has been. Reachability is a narrower exception (fleetd #315, below): a profile is never
|
||||||
|
* checked for reachability up front, only skipped once it has already failed in <em>this same</em>
|
||||||
|
* spawn call's retry loop — see the unreachable case below.
|
||||||
*
|
*
|
||||||
* <p>Four exceptions walk past the default instead of returning it unconditionally:
|
* <p>Four exceptions walk past the default instead of returning it unconditionally:
|
||||||
* <ul>
|
* <ul>
|
||||||
|
|||||||
Reference in New Issue
Block a user