[
https://issues.apache.org/jira/browse/GEODE-7956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120798#comment-17120798
]
ASF GitHub Bot commented on GEODE-7956:
---------------------------------------
dschneider-pivotal commented on a change in pull request #5159:
URL: https://github.com/apache/geode/pull/5159#discussion_r433083367
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/cache/RegionNameValidation.java
##########
@@ -24,7 +24,7 @@
public class RegionNameValidation {
- private static final Pattern NAME_PATTERN =
Pattern.compile("[aA-zZ0-9-_.]+");
+ private static final Pattern NAME_PATTERN =
Pattern.compile("[a-zA-Z\\[\\]0-9-_.]+");
Review comment:
When I read aA-zZ I thought it was just a special regex pattern to match
a-zA-Z. I bet whoever wrote this thought that. But since the product was
already released with a pattern that allows those other characters it would be
safest to continue to allow those characters. To make a breaking change I think
we need an RFC. For now it is best to change the docs to describe the behavior
of the existing geode release. We could schedule removing these special
characters from region names in a future release. I think it would be good to
rewrite the regex to still match what it did before but to do it more
explicitly (something like a-zA-Z0-9-_.^`[]\\) or add comments to make clear
what A-z matches. I also think it would be good to have some explicit test
methods that show that these special characters are allowed in region names.
Thanks for figuring this out!
----------------------------------------------------------------
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:
us...@infra.apache.org
> Correct documentation of legal region names
> -------------------------------------------
>
> Key: GEODE-7956
> URL: https://issues.apache.org/jira/browse/GEODE-7956
> Project: Geode
> Issue Type: Bug
> Components: docs
> Reporter: Darrel Schneider
> Assignee: Mario Kevo
> Priority: Major
>
> According to this:
> https://geode.apache.org/docs/guide/111/basic_config/data_regions/region_naming.html
> region names can only contain alphanumeric, dash, and underscore.
> But the product also supports a dot '.'.
> See RegionNameValidation and RegionNameValidationTest
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
