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/25 04:01:47 UTC

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

Technoboy- opened a new pull request #14858:
URL: https://github.com/apache/pulsar/pull/14858


   ### Motivation
   
   Users can create partitioned topics success even namespace does not exist via the V1 path.
   
   ### Documentation
   
   - [x] `no-need-doc` 
     
   


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

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

Posted by GitBox <gi...@apache.org>.
Technoboy- commented 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. 


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

[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

Posted by GitBox <gi...@apache.org>.
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 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

[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

Posted by GitBox <gi...@apache.org>.
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

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

Posted by GitBox <gi...@apache.org>.
Technoboy- closed pull request #14858:
URL: https://github.com/apache/pulsar/pull/14858


   


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