yuruguo opened a new pull request #12260: URL: https://github.com/apache/pulsar/pull/12260
### Motivation Currently, we use new option to replace some deprecated option in `client-tools`. for example: https://github.com/apache/pulsar/blob/9d309145f342bc416b8b4663125e1216903a3e83/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdSources.java#L141-L144 In order to maintain compatibility, the deprecated option still take effect through merging, as below: https://github.com/apache/pulsar/blob/9d309145f342bc416b8b4663125e1216903a3e83/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdSources.java#L183-L191 But I found that its priority is higher than the new option, which causes the new option to be invalid when we set both at the same time. ### Modifications Adjust the priority of the new and deprecated option, the deprecated option only takes effect when the new option is not set. ### Documentation Need to update docs? - doc-required -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
