[GitHub] [geode] jinmeiliao commented on pull request #3059: GEODE-6184 Added internal implementation of ClusterManagementService Java API to create a region

["jinmeiliao (GitHub)" <gi...@apache.org>]

if value is `parent/child`, then this would fail to catch this. Could change it to:

```
if (value == null) {
      throw new IllegalArgumentException("Region name cannot be null");
    }

    if (value.startsWith("/")) {
      value = value.substring(1);
    }

    if (value.split("/").length > 1) {
      throw new IllegalArgumentException("Region name is invalid -- cannot have multiple slashes");
    }
    
    this.name = value;
```

[ Full content available at: https://github.com/apache/geode/pull/3059 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org