hiSandog opened a new pull request, #25907: URL: https://github.com/apache/pulsar/pull/25907
Fixes: none ### Motivation `FieldParser` parses map configuration values by splitting every `=` character. This rejects values that contain `=` and also rejects empty string values such as `key=`, even though both are valid string map values after the first key/value separator. ### Modifications Split map entries on the first `=` only, preserving the rest of the value. Added a regression test covering a value that contains `=` and an empty map value. ### Verifying this change - [ ] Make sure that the change passes the CI checks. This change added tests and can be verified as follows: - Added `FieldParserTest.testMapWithEqualsSignAndEmptyValue` - Ran `git diff --check` - Attempted `./gradlew :pulsar-common:test --tests org.apache.pulsar.common.util.FieldParserTest`, but the local environment has no Java Runtime installed, so Gradle could not start. ### Does this pull request potentially affect one of the following parts: - [ ] 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 -- 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]
