Skip to content

[feat][broker] Expose managed ledger properties via topic internal stats - #26079

Merged
BewareMyPower merged 2 commits into
apache:masterfrom
BewareMyPower:bewaremypower/internal-stats-ml-properties
Jun 23, 2026
Merged

[feat][broker] Expose managed ledger properties via topic internal stats#26079
BewareMyPower merged 2 commits into
apache:masterfrom
BewareMyPower:bewaremypower/internal-stats-ml-properties

Conversation

@BewareMyPower

Copy link
Copy Markdown
Contributor

Motivation

Currently there is no way to check managed ledger properties unless reading directly from the metadata store or dumping the heap memory. The managed ledger properties could include the topic properties, or last index if AppendIndexMetadataInterceptor is configured, or a custom property set from downstream plugins via ManagedLedger#setProperty. It would be helpful if we can directly check the properties via admin tools like pulsar-shell.

Modifications

Add a properties field to managed ledger stats and fill it with a snapshot of propertiesMap. Add testGetInternalStatsWithProperties to verify the change.

@BewareMyPower BewareMyPower self-assigned this Jun 23, 2026
@BewareMyPower BewareMyPower added type/feature The PR added a new feature or issue requested a new feature area/broker release/4.2.3 release/4.0.12 labels Jun 23, 2026
@BewareMyPower
BewareMyPower merged commit 6d0d99b into apache:master Jun 23, 2026
43 checks passed
@BewareMyPower
BewareMyPower deleted the bewaremypower/internal-stats-ml-properties branch June 23, 2026 08:33
BewareMyPower added a commit that referenced this pull request Jun 23, 2026
BewareMyPower added a commit that referenced this pull request Jun 23, 2026
sandeep-ctds pushed a commit to datastax/pulsar that referenced this pull request Jul 31, 2026
@lhotari lhotari added this to the 5.0.0-M2 milestone Aug 14, 2026
nodece pushed a commit to ascentstream/pulsar that referenced this pull request Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants