Kouhei Sutou
kou****@clear*****
Fri Mar 27 14:34:23 JST 2015
> - logger.info("cluster state changed")
> - logger.info(@state)
> + logger.info("cluster state changed: #{JSON.generate(@state)}")
logger.info("cluster state changed", :state => @state)
ってしたら動くんじゃなかったでしたっけ。
In <c79ddf16d4dd0f7be57573b0f19f78f981b5c981 �� jenkins.clear-code.com>
"[Groonga-commit] droonga/droonga-engine �� c79ddf1 [master] Stringify loaded cluster state for logging by self." on Fri, 27 Mar 2015 14:29:02 +0900,
YUKI Hiroshi <null+groonga �� clear-code.com> wrote:
> YUKI Hiroshi 2015-03-27 14:29:02 +0900 (Fri, 27 Mar 2015)
>
> New Revision: c79ddf16d4dd0f7be57573b0f19f78f981b5c981
> https://github.com/droonga/droonga-engine/commit/c79ddf16d4dd0f7be57573b0f19f78f981b5c981
>
> Message:
> Stringify loaded cluster state for logging by self.
>
> logger.*() doesn't accept Hash objects.
>
> Modified files:
> lib/droonga/cluster.rb
>
> Modified: lib/droonga/cluster.rb (+1 -2)
> ===================================================================
> --- lib/droonga/cluster.rb 2015-03-27 14:23:58 +0900 (d8a648c)
> +++ lib/droonga/cluster.rb 2015-03-27 14:29:02 +0900 (b3d0ecf)
> @@ -101,8 +101,7 @@ module Droonga
> if @state == old_state
> logger.info("cluster state not changed")
> else
> - logger.info("cluster state changed")
> - logger.info(@state)
> + logger.info("cluster state changed: #{JSON.generate(@state)}")
> engine_nodes.each(&:resume)
> on_change
> end