davebarnes97 commented on a change in pull request #6509:
URL: https://github.com/apache/geode/pull/6509#discussion_r638122380
##########
File path: geode-docs/developing/geode_apis_compatible_with_redis.html.md.erb
##########
@@ -218,7 +184,7 @@ These commands are supported for Redis 5.
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 second. If they are due
to expire, they are deleted. Passive expiration is accurate to the second.
+- With passive expiration, keys are evaluated every 3 minutes. If they are
due to expire, they are deleted. Passive expiration is accurate to the second.
Review comment:
I agree with @nonbinaryprogrammer that "accurate to the second" most
likely refers to the resolution of the comparison between the present time and
the specified expiration time. Does the code support this interpretation?
I infer from @sabbey37 's comment that the docs could drop the "accurate to
..." clauses.
--
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:
us...@infra.apache.org
