Revert "types: support working with binary for Python 3"

This reverts commit a2d79da.

Author: DifferentialOrange

tarantool/request.py

@@ -4,7 +4,6 @@
 Request types definitions
 '''
 
-import sys
 import collections
 import msgpack
 import hashlib
@@ -85,13 +84,8 @@ def __init__(self, conn):
         # The option controls whether to pack binary (non-unicode)
         # string values as mp_bin or as mp_str.
         #
-        # The default behaviour of the Python 2 connector is to pack
-        # both bytes and Unicode strings as mp_str.
-        #
-        # The default behaviour of the Python 3 connector (since
-        # default encoding is "utf-8") is to pack bytes as mp_bin
-        # and Unicode strings as mp_str. encoding=None mode must
-        # be used to work with non-utf strings.
+        # The default behaviour of the connector is to pack both
+        # bytes and Unicode strings as mp_str.
         #
         # msgpack-0.5.0 (and only this version) warns when the
         # option is unset:
@@ -104,10 +98,7 @@ def __init__(self, conn):
         # just always set it for all msgpack versions to get rid
         # of the warning on msgpack-0.5.0 and to keep our
         # behaviour on msgpack-1.0.0.
-        if conn.encoding is None or sys.version_info.major == 2:
-            packer_kwargs['use_bin_type'] = False
-        else:
-            packer_kwargs['use_bin_type'] = True
+        packer_kwargs['use_bin_type'] = False
 
         self.packer = msgpack.Packer(**packer_kwargs)
 

tarantool/utils.py

@@ -6,10 +6,7 @@
 if sys.version_info.major == 2:
     string_types     = (basestring, )
     integer_types    = (int, long)
-    supported_types  = integer_types + string_types + (float,)
-
     ENCODING_DEFAULT = None
-
     if sys.version_info.minor < 6:
         binary_types = (str, )
     else:
@@ -20,13 +17,10 @@ def strxor(rhs, lhs):
         return "".join(chr(ord(x) ^ ord(y)) for x, y in zip(rhs, lhs))
 
 elif sys.version_info.major == 3:
-    binary_types    = (bytes, )
-    string_types    = (str, )
-    integer_types   = (int, )
-    supported_types = integer_types + string_types + binary_types + (float,)
-
+    binary_types  = (bytes, )
+    string_types  = (str, )
+    integer_types = (int, )
     ENCODING_DEFAULT = "utf-8"
-
     from base64 import decodebytes as base64_decode
 
     def strxor(rhs, lhs):
@@ -49,7 +43,7 @@ def check_key(*args, **kwargs):
         elif args[0] is None and kwargs['select']:
             return []
     for key in args:
-        assert isinstance(key, supported_types)
+        assert isinstance(key, integer_types + string_types + (float,))
     return list(args)