PR-20832: [improve][broker] Heartbeat topic enable checkMessageExpire

[streamnativebot]

url: apache#20832

Motivation

Running pulsar cluster for a long time, we found there are thousands of ledgers of heartbeat topic. The reason is heartbeat topic do not clean the expired message because it is systemTopic. The behaviour change since this pr apache#13611.

I think it is no need to keep heartbeat topic message permanently and we'd better clean the expired message and the ledger.

Modifications

modify checkMessageExpire()

Verifying this change

• Make sure that the change passes the CI checks.

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: