nonbinaryprogrammer commented on a change in pull request #6235:
URL: https://github.com/apache/geode/pull/6235#discussion_r607205527
##########
File path: geode-docs/tools_modules/geode_apis_compatible_with_redis.html.md.erb
##########
@@ -68,6 +68,7 @@ The <%=vars.product_name%> APIs compatible with Redis
currently supports the fol
**Note**: These commands are supported for Redis 5.
**Note**: SLOWLOG is implemented as a NoOp for compatibility with some
monitoring tools.
+**Note**: Redis accepts Longs as the parameters for the HSCAN command. The
Geode APIs compatible with Redis only accept Integer values. Parameters out of
the range of the 32-bit integer type will throw an exception.
Review comment:
I definitely agree with that but the current format doesn't make that
very easy. [this PR](https://github.com/apache/geode/pull/6239/) changes the
format of this list, which will allow an inline note. I will make this change
when I rebase that command on top of this (once it is merged)
--
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]
