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 2022/03/27 12:19:08 UTC

[GitHub] [pulsar] Technoboy- edited a comment on pull request #14858: [fix][pulsar-admin] Fix pulsar-admin could create topic even namespace does not exist via V1 path

Technoboy- edited a comment on pull request #14858:
URL: https://github.com/apache/pulsar/pull/14858#issuecomment-1079918908


   > I support this patch.
   > 
   > But...Is this kind of a breaking change?
   > 
   > @merlimat @rdhabalia PTAL
   
   I think we have missed some check
   
   > I support this patch.
   > 
   > But...Is this kind of a breaking change?
   > 
   > @merlimat @rdhabalia PTAL
   
   I also have the same question about this patch.  But push this patch because `createNonPartitionedTopic` has checked the namespace policy, but partitioned topic not.  And it's related to #9121


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