You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Darrel Schneider (Jira)" <ji...@apache.org> on 2021/10/26 21:38:00 UTC
[jira] [Updated] (GEODE-9776) geode-for-redis README.md can be
simplified
[ https://issues.apache.org/jira/browse/GEODE-9776?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Darrel Schneider updated GEODE-9776:
------------------------------------
Affects Version/s: 1.15.0
> geode-for-redis README.md can be simplified
> -------------------------------------------
>
> Key: GEODE-9776
> URL: https://issues.apache.org/jira/browse/GEODE-9776
> Project: Geode
> Issue Type: Improvement
> Components: redis
> Affects Versions: 1.15.0
> Reporter: Darrel Schneider
> Priority: Major
>
> The geode-for-redis README.md is more complicated than it needs to be.
> When starting a server for redis in gfsh I recommend just the following:
>
> {noformat}
> start server --geode-for-redis-port=<port> --disable-default-server=true
> {noformat}
> No need for --name. No need for --geode-for-redis-bind-address. No need for --locators.
> I suggest the "--disable-default-server=true" so that it will not also try to listen for geode clients on 40404 which can cause a failure if that port is already in use. This also prepares the way for the second server they start which will not need "--server-port=0". We don't need to tell them "If any of the options..." right after the start server. This is not a reference manual. Just tell them how to start the server.
> Also the README.md should mention that gfsh supports tab completion which can help with some of these long option names.
> The section that tells them to run redis-cli only needs to mention "-c -p <port>". No need for the "-h" since the server is listening on all local addresses. It should tell them to run redis-cli on the same machine as gfsh. Also it should mention (for rookies) that redis-cli is part of the native redis distribution which they may need to install. It does not come with geode. Also the README.md does not need to mention "<geodeForRedisPassword>" in the "Replace" sentence.
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)