-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Schema] Schema compatibility strategy in broker level. #11856
[Schema] Schema compatibility strategy in broker level. #11856
Conversation
doc = "The schema compatibility strategy in broker level. If this config in namespace policy is `UNDEFINED`" | ||
+ ", schema compatibility strategy check will use it in broker level." | ||
) | ||
private String schemaCompatibilityStrategy = "UNDEFINED"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we use the name of the enum SchemaCompatibilityStrategy.UNDEFINED
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good idea
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be able to directly use the enum type here
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractTopic.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for your contribution. For this PR, do we need to update docs? (The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks) |
@eolivelli This PR is already in branch-2.9 and will be released in 2.9.1. So I remove the label |
## Motivation link apache#11849 Schema compatibility strategy config in broker level. ## implement If namespace schema compatibility strategy is `UNDEFINED`, use broker schema compatibility strategy
Motivation
link #11849
Schema compatibility strategy config in broker level.
implement
If namespace schema compatibility strategy is
UNDEFINED
, use broker schema compatibility strategyVerifying this change
Add the tests for it
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)