[
https://issues.apache.org/jira/browse/GEODE-8141?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17113360#comment-17113360
]
ASF GitHub Bot commented on GEODE-8141:
---------------------------------------
DonalEvans closed pull request #5127:
URL: https://github.com/apache/geode/pull/5127
----------------------------------------------------------------
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
> Move definition of Region separator character to geode-common and make usage
> consistent throughout the codebase
> ---------------------------------------------------------------------------------------------------------------
>
> Key: GEODE-8141
> URL: https://issues.apache.org/jira/browse/GEODE-8141
> Project: Geode
> Issue Type: Task
> Reporter: Donal Evans
> Priority: Major
>
> The separator character for region paths is currently defined in the Region
> interface in geode-core. However, geode-management, which does not have a
> dependency on geode-core, uses the separator in several classes.
> * The definitions of the separator character and string should be moved to
> the geode-common module.
> * The currently existing definitions of the separator in the geode-core
> Region interface should be deprecated.
> * All instances of hardcoded '/' characters and strings should be replaced
> with references to the constants in geode-common.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
