pulsar
pulsar copied to clipboard
[cleanup][broker] Add Maven Modernizer plugin in pulsar-broker and fix violation in `test: broker/admin`
Master Issue: #12271 #16991
Motivation
Apply Maven Modernizer plugin to enforce we move away from legacy APIs.
Modifications
Add Maven Modernizer plugin in pulsar-broker module 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:
- 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
- [x]
doc-not-needed
Related to #16991 ?
Related to #16991 ?
yes, I have fixed the reference.
/pulsarbot run-failure-checks
/pulsarbot run-failure-checks
/pulsarbot run-failure-checks
@youzipi It seems one required test failed. Please merge master and push for another run.
/pulsarbot run-failure-checks
/pulsarbot run-failure-checks
how to trigger the canceled
checks? @tisonkun
i think this command should work , according to https://github.com/apache/pulsar-test-infra/blob/master/pulsarbot/entrypoint.sh
@youzipi https://github.com/apache/pulsar/pull/16993#issuecomment-1219396221 this comment is correct, due to CI traffic pulsarbot may queue for resources to execute. Wait a minute :)
Thanks for your contribution @youzip! Go ahead.