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 2018/04/06 16:47:27 UTC

[GitHub] merlimat opened a new pull request #1516: Added test to ensure property name can contain '_' characters.

merlimat opened a new pull request #1516: Added test to ensure property name can contain '_' characters.
URL: https://github.com/apache/incubator-pulsar/pull/1516
 
 
   ### Motivation
   
   As reported in #1506, in 1.22 we still have inconsistencies between what's allowed as a valid property name and what is allowed when creating the namespace. 
   
   This issue was already fixed in master. `_` are allowed in property name and invalid characters are correctly rejected when creating the property.
   
   Adding here tests to ensure the behavior doesn't change.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services