[GitHub] [pulsar] codelipenghui commented on pull request #14158: [pulsar-broker] Fix: return non-null persistence policy for namespace

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

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


   > I think we can add an `applied` to control whether we should return namespace setting or the value that actually take effects, like many other polices in `org.apache.pulsar.broker.admin.v2.PersistentTopics`
   
   Agree, with this change, users will not able to know what is the namespace settings. We should apply the same behavior, for policies, if the topic/namespace don't have a policy, just provide a response to tell users no policy for this topic/namespace, if users want to check which policy will be applied, they can add --applied`.


-- 
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