[improve][broker] add warn log when the delayedMessagesCount is incorrect.

[thetumbled]

Motivation

#24430 (comment)

Add check for whether the delayedMessagesCount is correct.

Modifications

Add warn log.

Verifying this change

• Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository:

[codelipenghui]

/pulsarbot run-failure-checks

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 74.28%. Comparing base (bbc6224) to head (ff519ef).
Report is 1181 commits behind head on master.

Files with missing lines	Patch %	Lines
...broker/delayed/InMemoryDelayedDeliveryTracker.java	0.00%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #24460      +/-   ##
============================================
+ Coverage     73.57%   74.28%   +0.70%     
- Complexity    32624    32737     +113     
============================================
  Files          1877     1868       -9     
  Lines        139502   145505    +6003     
  Branches      15299    16644    +1345     
============================================
+ Hits         102638   108087    +5449     
+ Misses        28908    28856      -52     
- Partials       7956     8562     +606     

Flag	Coverage Δ
inttests	26.87% <0.00%> (+2.28%)	⬆️
systests	23.35% <0.00%> (-0.98%)	⬇️
unittests	73.76% <0.00%> (+0.91%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...broker/delayed/InMemoryDelayedDeliveryTracker.java	88.77% <0.00%> (+0.63%)	⬆️

... and 1080 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.