[GitHub] [pulsar] codelipenghui commented on pull request #14409: Make sure policies.is_allow_auto_update_schema not null

[GitBox <gi...@apache.org>]

codelipenghui commented on pull request #14409:
URL: https://github.com/apache/pulsar/pull/14409#issuecomment-1047742951


   > We can add a new interface Policies getPolicies(String namespace, boolean applied) in org.apache.pulsar.client.admin.Namespaces. And set applied as true by default in getPolicies(String namespace).
   
   the applied option should be false by default?  to keep consistent with topic policy does. It's better to start with a PIP, as far as I understand, not all the namespace policies follow the same way, some return null if the namespace doesn't have the policy, some return value from broker level, we need to improve this situation even if the inevitable break change may be introduced, otherwise, we will always be confused by different behaviors.
   
   We should add a release notice for https://github.com/apache/pulsar/pull/12786


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org