Skip to content

Conversation

@MarvinCai
Copy link
Contributor

Master Issue: #12271

Motivation

Apply Maven Modernizer plugin to enforce we move away from legacy APIs for pulsar-client modeules.

Modifications

Add Maven Modernizer plugin in pom and fix violation.

Verifying this change

This change is already covered by existing tests, such as (please describe tests).

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

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

Check the box below and label this PR (if you have committer privilege).

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Nov 25, 2021
@codelipenghui codelipenghui added this to the 2.10.0 milestone Nov 25, 2021
@merlimat merlimat merged commit 656398e into apache:master Nov 29, 2021
fxbing pushed a commit to fxbing/pulsar that referenced this pull request Dec 19, 2021
@BewareMyPower
Copy link
Contributor

Add the release/2.9.3 label since this PR is depended by #15413

@BewareMyPower
Copy link
Contributor

This PR has changed too many files, remove the release/2.9.3 label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants