Permalink
Browse files

fix logging

  • Loading branch information...
ryanlecompte committed Nov 14, 2012
1 parent 98194ad commit f7a4d8127ad86dbf3ea84b70af32fb071f845be5
Showing with 2 additions and 1 deletion.
  1. +1 −0 Changes.md
  2. +1 −1 lib/redis_failover/node_manager.rb
View
@@ -1,6 +1,7 @@
HEAD
-----------
- Bumped required dependency on ZK gem. ZK 1.7.4 fixes a critical bug with locking (see https://github.com/slyphon/zk/issues/54)
+- Fix an issue where a failover would not occur if we couldn't check the role of a downed master
1.0.0
-----------
@@ -264,7 +264,7 @@ def find_existing_master
raise InvalidNodeRoleError.new(master, :master, :slave)
end
rescue RedisFailover::NodeUnavailableError => ex
- logger.warn("Failed to check whether existing master has invalid role", ex)
+ logger.warn("Failed to check whether existing master has invalid role: #{ex.inspect}")
end
master

0 comments on commit f7a4d81

Please sign in to comment.