Skip to content
This repository was archived by the owner on Apr 1, 2024. It is now read-only.
This repository was archived by the owner on Apr 1, 2024. It is now read-only.

PR-20833: [improve][broker] Add bookkeeperDeleted in topic internal stats. #5836

@streamnativebot

Description

@streamnativebot

url: apache#20833

Motivation

Shows the offload detail of the ledgers whether it is deleted in bookkeeper.

The ./bin/pulsar-admin topics stats-internal command now shows the ledger info of specific topic. If we are using offloader, we can know the offloaded of the ledger but not be able to know whether it is deleted in bookkeeper.

image

Modifications

Add bookkeeperDeleted field in org.apache.pulsar.common.policies.data.ManagedLedgerInternalStats.LedgerInfo.

Fill the bookkeeperDeleted in all places where offloaded be filled.

Verifying this change

  • Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

  • org.apache.pulsar.tests.integration.offload.TestBaseOffload#testPublishOffloadAndConsumeDeletionLag

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: dragonls#11

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions