Skip to content

Commit 76881e6

Browse files
committed
Added justification for restraint on borrowing
1 parent 6224f2a commit 76881e6

File tree

1 file changed

+15
-0
lines changed
  • keps/sig-api-machinery/1040-priority-and-fairness

1 file changed

+15
-0
lines changed

keps/sig-api-machinery/1040-priority-and-fairness/README.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -708,6 +708,21 @@ existing systems will be more continuous if we keep the meaning of
708708
"total shares" for the existing field. In the next version we should
709709
rename the `AssuredConcurrencyShares` to `NominalConcurrencyShares`.
710710

711+
The limits on borrowing are two-sided: a given priority level has a
712+
limit on how much it may borrow and a limit on how much may be
713+
borrowed from it. The latter is a matter of protection, the former is
714+
a matter of restraint. Introducing just the protection side is not
715+
enough. If APF gained borrowing without giving the cluster
716+
administrators a way to limit how much a given priority level borrows
717+
then the administrators would not be able to do something they could
718+
do before the introduction of borrowing: use a priority level as a
719+
deliberate jail for some class of traffic that APF is not limiting
720+
well. APF dispatches requests based on approximate estimates of how
721+
much work they involve. We have been improving these estimates, and
722+
may continue to do so, but there will always remain the possibility
723+
that some class of requests is much "heavier" than the APF code
724+
estimates; for those, a deliberate jail is useful.
725+
711726
The following table shows the current default non-exempt priority
712727
levels and a proposal for their new configuration.
713728

0 commit comments

Comments
 (0)