[GitHub] [geode] jdeppe-pivotal commented on a change in pull request #6960: GEODE-9562: add system property for redis region name

[GitBox <gi...@apache.org>]

jdeppe-pivotal commented on a change in pull request #6960:
URL: https://github.com/apache/geode/pull/6960#discussion_r725127784



##########
File path: geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
##########
@@ -1978,12 +1966,18 @@
    * <U>Default</U>: 6379
    * </p>
    * <U>Allowed values</U>: 0..65535
-   *
-   * <p>
-   * Experimental: Geode API compatible with Redis is subject to change in future releases
-   * <p/>
    */
   String REDIS_PORT = "geode-for-redis-port";
+  /**
+   * The static String definition of the <i>"geode-for-redis-region-name"</i> property <a
+   * name="geode-for-redis-region-name"/a>
+   * </p>
+   * <U>Description</U>: Specifies the name of the region for Geode for Redis data</td>
+   * </p>
+   * <U>Default</U>: REDIS_DATA
+   * </p>
+   */
+  String REDIS_REGION_NAME = "geode-for-redis-region-name";

Review comment:
       This should just be defined in `RedisConstants`.




-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org