gaoran10 opened a new pull request #11595:
URL: https://github.com/apache/pulsar/pull/11595
Fixes #11533
### Motivation
Refer to issue #11533 , currently, if users redefined the same `Record` or
`Enum` in `Record`, the schema info isn't reused the defined name, this does
not match the Avro schema info format.
### Modifications
Add a new method `schema_info(self, defined_names)` in `Record`, `Array`,
`Map`, and `Enum`, all defined names will be added in the parameter
`defined_names` when users use a defined `Record`, or `Enum`, the schema info
will use the name of the defined `Record` or `Enum` as the type.
### Verifying this change
Add test to verify using same `Record` or `Enum` in a complex `Record`.
### Does this pull request potentially affect one of the following parts:
*If `yes` was chosen, please highlight the changes*
- 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
#### For committer
For this PR, do we need to update docs?
- If yes,
- if you update docs in this PR, label this PR with the `doc` label.
- if you plan to update docs later, label this PR with the `doc-required`
label.
- if you need help on updating docs, create a follow-up issue with the
`doc-required` label.
- If no, label this PR with the `no-need-doc` label and explain why.
--
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]
