The GitHub Actions job "License checker" on 
rocketmq.git/fix/batch-cq-estimate-count-boundary has failed.
Run started by GitHub user zjncs (triggered by zjncs).

Head commit for run:
a310aed3ce13551c2dba613d6c63e891dffd3dc4 / zjncs <[email protected]>
fix(store): estimate batch consume queue count up to max offset

estimateMessageCount translated the exclusive upper bound 'to' with
getBatchMsgIndexBuffer(to), which returns null when 'to' equals
maxOffsetInQueue. Every accumulation query that counts up to the queue
head therefore hit the null branch and returned -1, and callers such as
DefaultMessageStore silently fell back to the raw offset difference,
ignoring batch sizes and tag filters.

Since 'to' is exclusive, look up the unit containing 'to - 1' instead
and advance physicalOffsetTo past that unit so the scan covers it.
Interior ranges are unaffected because a 'to' that lands on a batch base
resolves to the same mapped position as before.

Report URL: https://github.com/apache/rocketmq/actions/runs/34573483654

With regards,
GitHub Actions via GitBox

Reply via email to