[
https://issues.apache.org/jira/browse/GEODE-7956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17119946#comment-17119946
]
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_r432739787
##########
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:
I don't understand why you changed this Pattern. I think GEODE-7956 was
just about correcting the docs. It looks to me like you are expanding what a
legal region name is to include square brackets. Did this get approved in a
RFC? If not I think you should do so before this gets approved. In we end up
keeping square brackets in region names then update GEODE-7956 to say we should
and why it is valuable.
----------------------------------------------------------------
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:
[email protected]
> 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)
