[
https://issues.apache.org/jira/browse/GEODE-5609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16587772#comment-16587772
]
ASF subversion and git services commented on GEODE-5609:
--------------------------------------------------------
Commit 49e1c18ad4d20f3d5c468f4b69a2debe2a6382f3 in geode's branch
refs/heads/develop from [~apa...@the9muses.net]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=49e1c18 ]
GEODE-5609: Extract validateRegionName and improve testing (#2353)
* Extract validateRegionName from LocalRegion to RegionNameValidation
* Move and rename RegionNameValidationJUnitTest
* Rewrite RegionNameValidationTest with AssertJ and modern testing
> Region names should be able to contain multiple underscores
> -----------------------------------------------------------
>
> Key: GEODE-5609
> URL: https://issues.apache.org/jira/browse/GEODE-5609
> Project: Geode
> Issue Type: Bug
> Components: regions
> Reporter: Kirk Lund
> Assignee: Kirk Lund
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> Two leading underscores is not allowed (this is reserved for internal region
> names in Geode).
> However, multiple underscores later in the region name is supposed to be ok.
> Attempting to use more than one underscore results in this failure:
> {noformat}
> Caused by: java.lang.IllegalArgumentException: Region names may only be
> alphanumeric and may contain hyphens or underscores: class
> org.apache.geode.management.RegionManagementDUnitTest_testDistributedRegion_region
> at
> org.apache.geode.internal.cache.LocalRegion.validateRegionName(LocalRegion.java:7429)
> at
> org.apache.geode.internal.cache.GemFireCacheImpl.createVMRegion(GemFireCacheImpl.java:3003)
> at
> org.apache.geode.internal.cache.GemFireCacheImpl.basicCreateRegion(GemFireCacheImpl.java:2982)
> at
> org.apache.geode.internal.cache.GemFireCacheImpl.createRegion(GemFireCacheImpl.java:2970)
> at org.apache.geode.cache.RegionFactory.create(RegionFactory.java:756)
> {noformat}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
