kaijaytu commented on PR #570:
URL: https://github.com/apache/yunikorn-site/pull/570#issuecomment-5326344616
Follow-up status for the two items @manirajv06 agreed to spin off:
1. Design doc quota_preemptor.md still shows the three mismatches with
implementation on current master (66ef06c):
- Timer behavior: doc says "timers reset and start again from the
beginning", but the code uses delta adjustment (startTime += newDelay
- oldDelay) for delay changes and preserves the earliest startTime
for consecutive quota decreases.
- Delay format and placement: doc shows an integer in seconds under the
resources: block (e.g. quota.preemption.delay: 7200); implementation
uses Go time.Duration under queue properties:.
- Impact of Restart: doc only says the timer "would start again from the
beginning" and does not mention that IncAllocatedResource() during
recovery triggers setPreemptionTime() when usage > max.
2. Pre-existing broken anchors: no commits have touched versioned_docs/
since #570 was closed, so the ~50 broken anchors (mostly in 1.0.0 to
1.3.0) are still present.
I will file two new JIRAs and link them here once created.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
