sabbey37 commented on a change in pull request #6235:
URL: https://github.com/apache/geode/pull/6235#discussion_r605971134
##########
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:
Forgot to mention, @karensmolermiller previously made a comment in [this
PR](https://github.com/apache/geode/pull/6149) regarding the command notes:
```
If there will be more notes about specific commands, I'd like to see the
notes placed next to the
command within the lists. Maybe add the note in parenthesis? I believe that
documentation readers are
more likely to see the note if it is with the command, as I believe that
readers will be likely to scan
this page for the commands that they use. I don't think they'll be starting
at the top to do a detailed
reading.
```
--
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]
