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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
