[
https://issues.apache.org/jira/browse/GEODE-8122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17108463#comment-17108463
]
ASF GitHub Bot commented on GEODE-8122:
---------------------------------------
sabbeyPivotal commented on a change in pull request #5122:
URL: https://github.com/apache/geode/pull/5122#discussion_r425932606
##########
File path:
geode-redis/src/main/java/org/apache/geode/redis/internal/GeodeRedisServer.java
##########
@@ -93,65 +93,11 @@
* server, each command is picked up by a thread, interpreted and then
executed and a response is
* sent back to the client. The default connection port is 6379 but that can
be altered when run
* through GFSH or started through the provided static main class.
- * <p>
- * Each Redis data type instance is stored in a separate {@link Region} except
for the Strings and
- * HyperLogLogs which are collectively stored in one Region respectively. That
Region along with a
- * meta data region used internally are protected so the client may not store
keys with the name
- * {@link GeodeRedisServer#REDIS_META_DATA_REGION} or {@link
GeodeRedisServer#STRING_REGION}. The
- * default Region type is {@link RegionShortcut#PARTITION_REDUNDANT}. If the
- * {@link GeodeRedisServer#NUM_THREADS_SYS_PROP_NAME} system property is set
- * to 0, one thread per client will be created. Otherwise a worker thread pool
of specified size is
- * used or a default size of {@link Runtime#availableProcessors()} if the
property is not set.
- * <p>
- * Setting the AUTH password requires setting the property "redis-password"
just as "redis-port"
- * would be in xml or through GFSH.
- * <p>
- * The supported commands are as follows:
- * <p>
- * Supported String commands - APPEND, BITCOUNT, BITOP, BITPOS, DECR, DECRBY,
GET, GETBIT, GETRANGE,
- * GETSET, INCR, INCRBY, INCRBYFLOAT, MGET, MSET, MSETNX, PSETEX, SET, SETBIT,
SETEX, SETNX, STRLEN
- * <p>
- * Supported List commands - LINDEX, LLEN, LPOP, LPUSH, LPUSHX, LRANGE, LREM,
LSET, LTRIM, RPOP,
- * RPUSH, RPUSHX
- * <p>
- * Supported Hash commands - HDEL, HEXISTS, HGET, HGETALL, HINCRBY,
HINCRBYFLOAT, HKEYS, HMGET,
- * HMSET, HSETNX, HLEN, HSCAN, HSET, HVALS
- * <p>
- * Supported Set commands - SADD, SCARD, SDIFF, SDIFFSTORE, SINTER,
SINTERSTORE, SISMEMBER,
- * SMEMBERS, SMOVE, SREM, SPOP, SRANDMEMBER, SCAN, SUNION, SUNIONSTORE
- * <p>
- * Supported SortedSet commands - ZADD, ZCARD, ZCOUNT, ZINCRBY, ZLEXCOUNT,
ZRANGE, ZRANGEBYLEX,
- * ZRANGEBYSCORE, ZRANK, ZREM, ZREMRANGEBYLEX, ZREMRANGEBYRANK,
ZREMRANGEBYSCORE, ZREVRANGE,
- * ZREVRANGEBYSCORE, ZREVRANK, ZSCAN, ZSCORE
- * <p>
- * Supported HyperLogLog commands - PFADD, PFCOUNT, PFMERGE
- * <p>
- * Supported Keys commands - DEL, DBSIZE, EXISTS, EXPIRE, EXPIREAT, FLUSHDB,
FLUSHALL, KEYS,
- * PERSIST, PEXPIRE, PEXPIREAT, PTTL, SCAN, TTL
- * <p>
- * Supported Transaction commands - DISCARD, EXEC, MULTI
- * <p>
- * Supported Server commands - AUTH, ECHO, PING, TIME, QUIT
- * <p>
- * <p>
- * The command executors are not explicitly documented but the functionality
can be found at
- * <a href="http://redis.io/commands";>Redis Commands</a>
- * <p>
- * Exceptions to the Redis Commands Documents:
- * <p>
- * <ul>
- * <li>Any command that removes keys and returns a count of removed entries
will not return a total
- * remove count but rather a count of how many entries have been removed that
existed on the local
- * vm, though all entries will be removed</li>
- * <li>Any command that returns a count of newly set members has an
unspecified return value. The
- * command will work just as the Redis protocol states but the count will not
necessary reflect the
- * number set compared to overridden.</li>
- * <li>Transactions work just as they would on a Redis instance, they are
local transaction.
- * Transactions cannot be executed on data that is not local to the executing
server, that is on a
- * partitioned region in a different server instance or on a persistent region
that does not have
- * transactions enabled. Also, you cannot watch or unwatch keys as all keys
within a GemFire
- * transaction are watched by default.</li>
- * </ul>
+ *
+ * @See <a href=
+ *
"https://geode.apache.org/docs/guide/12/tools_modules/redis_api_for_geode.html";>Redis
API
Review comment:
Sounds good, I'll remove it.
----------------------------------------------------------------
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:
[email protected]
> Update GeodeRedisServer java doc and Redis API for Geode documentation
> ----------------------------------------------------------------------
>
> Key: GEODE-8122
> URL: https://issues.apache.org/jira/browse/GEODE-8122
> Project: Geode
> Issue Type: Improvement
> Components: redis
> Reporter: Sarah Abbey
> Priority: Major
>
> - Java doc for GeodeRedisServer updated for current GA release plans
> - Java doc for ConfigurationProperties updated so it doesn't reference
> internal GeodeRedisServer
> - Geode docs for Redis API for Geode updated
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
