mattisonchao opened a new pull request, #25193: URL: https://github.com/apache/pulsar/pull/25193
### Motivation After the PR https://github.com/apache/pulsar/pull/24617. We upgraded to a newer Avro version, which introduced breaking changes for our use case. The discussion context is here: https://lists.apache.org/thread/3xx7rx8571bbxc6c5s2ldbnmr06c42x3 This PR is only working to avoid breaking existing users. For the root cause of the issue, we should fix it in the Avro protobuf. ### Modifications - Support a `CompatibleNameValidator` instead of the default UTF_VALIDATOR to allow '$' to pass our validation. - Add a test to reproduce the issue. - Add a test for CompatibleNameValidator ### Verifying this change - [x] Make sure that the change passes the CI checks. ### Documentation - [x] `doc-not-needed` <!-- Your PR changes do not impact docs --> -- 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]
