Merge pull request holland-backup#145 from abg/revert_show_slave_stat…

…us_binary

Revert broken SHOW SLAVE STATUS fix for LP#1220841

CHANGES.rst

@@ -8,6 +8,18 @@ http://bugs.launchpad.net/holland-backup
 GH# referes to the deprecated github bug tracker here:
 https://github.com/holland-backup/holland/issues
 
+1.0.14 - unreleased
+-------------------
+
+holland
++++++++
+
+- A fix for launchpad bug #1220841 caused plugins that used
+  "SHOW SLAVE STATUS" via the holland mysql lib to fail with
+  an "unknown encoding: binary" error.  The changes for 
+  LP #1220841 have been reverted.
+
+
 1.0.12 - Feb 8, 2016
 --------------------
 

plugins/holland.lib.mysql/holland/lib/mysql/client/base.py

@@ -322,10 +322,8 @@ def show_slave_status(self):
         :returns: slave status dict
         """
         sql = "SHOW SLAVE STATUS"
-        charset = self.character_set_name()
         cursor = self.cursor()
         try:
-            self.set_character_set('binary')
             cursor.execute(sql)
             keys = [col[0].lower() for col in cursor.description]
             slave_status = cursor.fetchone()
@@ -334,14 +332,8 @@ def show_slave_status(self):
             if not slave_status:
                 return None
             else:
-                result = []
-                for value in slave_status:
-                    if isinstance(value, basestring):
-                        value = value.decode(charset, 'ignore')
-                    result.append(value)
-                return dict(zip(keys, result))
+                return dict(zip(keys, slave_status))
         finally:
-            self.set_character_set(charset)
             cursor.close()
 
     def show_master_status(self):