You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/05/09 01:54:28 UTC

[GitHub] [pulsar] codelipenghui edited a comment on pull request #6920: [Issue 6887][pulsar-client-tools] ttlDurationDefaultInSeconds not applying

codelipenghui edited a comment on pull request #6920:
URL: https://github.com/apache/pulsar/pull/6920#issuecomment-626086632


   @Omega-Ariston I think you can fix the `mergeNamespaceWithDefaults` method in the AdminResource class, we merge all namespace policy with the broker configuration here. And please add some unit tests for the change to avoid some other changes to break it.


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

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