nodece commented on a change in pull request #13938:
URL: https://github.com/apache/pulsar/pull/13938#discussion_r792278699
##########
File path:
pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/Policies.java
##########
@@ -100,11 +100,10 @@
@SuppressWarnings("checkstyle:MemberName")
@Deprecated
- public SchemaAutoUpdateCompatibilityStrategy
schema_auto_update_compatibility_strategy =
- SchemaAutoUpdateCompatibilityStrategy.Full;
+ public SchemaAutoUpdateCompatibilityStrategy
schema_auto_update_compatibility_strategy = null;
@SuppressWarnings("checkstyle:MemberName")
- public SchemaCompatibilityStrategy schema_compatibility_strategy =
SchemaCompatibilityStrategy.UNDEFINED;
+ public SchemaCompatibilityStrategy schema_compatibility_strategy = null;
Review comment:
Get Schema compatibility strategy of namespace API has been the break
that doesn't read broker level, so I think we can set `null` as
`schema_compatibility_strategy` value. When
`schema_auto_update_compatibility_strategy` will deleted, this API still
returns `null`.
--
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]
