[sheepdog] [PATCH v2] sheep/cluster: fix error in sheep log
Hitoshi Mitake
mitake.hitoshi at lab.ntt.co.jp
Thu Aug 7 07:43:45 CEST 2014
At Wed, 6 Aug 2014 11:47:39 +0800,
Bingpeng Zhu wrote:
>
> zk_timeout is measured in milliseconds, so
> it should be divided by 1000 to seconds.
>
> Signed-off-by: Bingpeng Zhu <bingpeng.zbp at alibaba-inc.com>
> ---
> sheep/cluster/zookeeper.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/sheep/cluster/zookeeper.c b/sheep/cluster/zookeeper.c
> index 5c181e1..fea8258 100644
> --- a/sheep/cluster/zookeeper.c
> +++ b/sheep/cluster/zookeeper.c
> @@ -971,7 +971,7 @@ static int zk_join(const struct sd_node *myself,
> rc = zk_node_exists(path);
> if (rc == ZOK) {
> sd_err("Previous zookeeper session exist, shoot myself. Please "
> - "wait for %d seconds to join me again.", zk_timeout);
> + "wait for %d seconds to join me again.", zk_timeout / 1000);
# Ccing Ruoyu, because he is a maintainer of zookeeper components.
If zk_timeout is less than 1000, the log will looks like "wait for 0
seconds ..." and it would be confusing for users. How about changing
the message like this: "wait for %d milli seconds"?
Ruoyu, do you have any comments?
Thanks,
Hitoshi
More information about the sheepdog
mailing list