michaeljmarshall opened a new pull request, #16862: URL: https://github.com/apache/pulsar/pull/16862
### Motivation Subscription properties are a new feature coming in 2.11. The admin api will allow for properties with commas, but the admin cli tool splits parameters with commas into multiple properties. This PR switches the splitter for the `properties` field and makes it possible to specify any valid property with the cli tool. Note: there are other properties fields that the CLI tool allows you to set, but because those have already been released, I am not updating those here. ### Modifications * Use the `NoSplitter.class` as the `splitter` for the subscription `properties` parameter. * Add test coverage for all changes. ### Verifying this change This PR includes new test coverage. ### Does this pull request potentially affect one of the following parts: This PR updates a CLI tool method that has not yet been released, so it does not break any existing functionality. ### Documentation - [x] `doc-not-needed` -- 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]
