DonalEvans commented on a change in pull request #7326:
URL: https://github.com/apache/geode/pull/7326#discussion_r794774023
##########
File path: geode-docs/tools_modules/geode_for_redis.html.md.erb
##########
@@ -229,27 +233,12 @@ integer (+/- 9223372036854775807) for CURSOR.
| server | redis_version<br/>redis_mode (always returns "cluster"
because <%=vars.product_name%> for Redis always runs in cluster
mode.)<br/>tcp_port<br/>uptime_in_seconds<br/>uptime_in_days |
| stats |
total_commands_processed<br/>instantaneous_ops_per_sec<br/>total_net_input_bytes<br/>instantaneous_input_kbps<br/>total_connections_received<br/>keyspace_hits<br/>keyspace_misses<br/>evicted_keys
(always returns 0)<br/>rejected_connections (always returns
0)<br/>pubsub_channels<br/>pubsub_patterns|
-## <a id="advantages-over-redis"></a>Advantages of <%=vars.product_name%> over
Redis
-
-<%=vars.product_name%>’s primary advantage is its **scalability**. While the
Redis server is single threaded, <%=vars.product_name%> supports high
concurrency. Many Redis clients can execute commands on the
<%=vars.product_name%> cluster simultaneously.
-
-<%=vars.product_name%>'s architecture and management features help detect and
resolve **network partitioning** problems without explicit management on the
part of the Redis client.
-
-<%=vars.product_name%> for Redis partitions data across multiple servers and
keeps replicated data up to date _synchronously_, whereas Redis uses
asynchronous replication.
-This provides a higher level of data consistency within the cluster.
-
## <a id="expiration-accuracy"></a>Expiration Accuracy
Keys are expired in two ways, actively and passively:
-- With active expiration, expiration is evaluated whenever a key is
accessed. If the key is due to expire, it is deleted. Active expiration is
accurate to the millisecond.
-- With passive expiration, keys are evaluated every three minutes. If they
are due to expire, they are deleted. Passive expiration is accurate to the
second.
-
-## <a id="high-availability-model"></a>High Availability Model
-
-Data is stored in a single partitioned region that by default has one
redundant copy.
-In practice this means that the cluster can tolerate the loss of a single
server without the loss of
-data.
+- With active expiration, expiration is evaluated **whenever a key is
accessed**. If the key is due to expire, it is deleted. Active expiration is
accurate to the millisecond.
+- With passive expiration, keys are evaluated **every three minutes**. If
they are due to expire, they are deleted. Passive expiration is accurate to
the second.
Review comment:
I'm not certain about the "Passive expiration is accurate to the second"
here. What does this actually mean? If we're only checking for passive
expiration every three minutes, it seems like we're only accurate to within
three minutes? Maybe this would be better removed?
##########
File path: geode-docs/tools_modules/geode_for_redis.html.md.erb
##########
@@ -17,25 +17,22 @@ limitations under the License.
<% set_title(product_name, "for Redis") %>
-<%=vars.product_name%> for Redis allows <%=vars.product_name%> to function as
a drop-in replacement for a
+<%=vars.product_name%> for Redis allows <%=vars.product_name%> to function as a
highly-available Redis data store, letting Redis applications take advantage of
-<%=vars.product_name%>’s scaling capabilities without changing their client
code. Redis clients connect to a <%=vars.product_name%>
-server in the same way they connect to a Redis server, using a hostname and a
port number, with
-optional password authentication.
+<%=vars.product_name%>’s scaling capabilities with minimal changes to their
client code.
-<img src="../images_svg/geode_for_redis.svg" class="image" />
## <a id="using-geode-for-redis"></a>Using <%=vars.product_name%> for Redis
+To use the <%=vars.product_name%> for Redis feature, the cluster must have at
least one server that has enabled the feature to handle incoming Redis commands.
-The <%=vars.product_name%> cluster must have at least one server that is set
up to handle the incoming Redis commands.
-
-Prerequisites for running the examples:
+Prerequisites:
1. **Install <%=vars.product_name%>** <br/>
Using the instructions in the `README.md` file in the root of the
<%=vars.product_name%> checkout directory, build and install
<%=vars.product_name%>.
2. **Install the Redis CLI** <br/>
Follow installation instructions at https://redis.io/download
Review comment:
This wording might make it sound like a prerequisite for using
geode-for-redis is to have the Redis CLI installed, which is not strictly true,
it's just used in this example to ping the server. Maybe this could be changed
to something along the lines of "have a Redis client so you can connect to the
server" for the prerequisite, and then say "in this example we'll be using the
Redis CLI, which can be installed from here..."?
--
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]
