BUG: Fix passing non-existent file to read_msgpack pandas-dev#15296

Author: chrisburr

pandas/io/packers.py

@@ -61,7 +61,8 @@
 from pandas.core.sparse.array import BlockIndex, IntIndex
 from pandas.core.generic import NDFrame
 from pandas.errors import PerformanceWarning
-from pandas.io.common import get_filepath_or_buffer, _stringify_path
+from pandas.io.common import (get_filepath_or_buffer, _get_handle,
+                              _stringify_path)
 from pandas.core.internals import BlockManager, make_block, _safe_reshape
 import pandas.core.internals as internals
 
@@ -186,39 +187,31 @@ def read_msgpack(path_or_buf, encoding='utf-8', iterator=False, **kwargs):
         return Iterator(path_or_buf)
 
     def read(fh):
+        print(fh)
         l = list(unpack(fh, encoding=encoding, **kwargs))
         if len(l) == 1:
             return l[0]
         return l
 
-    # see if we have an actual file
     if isinstance(path_or_buf, compat.string_types):
-
         try:
             exists = os.path.exists(path_or_buf)
+
+        # if the filepath is too long will raise here
+        # 5874
         except (TypeError, ValueError):
             exists = False
 
         if exists:
             with open(path_or_buf, 'rb') as fh:
                 return read(fh)
-
-    # treat as a binary-like
-    if isinstance(path_or_buf, compat.binary_type):
-        fh = None
-        try:
-            fh = compat.BytesIO(path_or_buf)
-            return read(fh)
-        finally:
-            if fh is not None:
-                fh.close()
-
-    # a buffer like
-    if hasattr(path_or_buf, 'read') and compat.callable(path_or_buf.read):
+        elif hasattr(path_or_buf, 'read'):
+            return read(path_or_buf)
+        else:
+            raise ValueError('path_or_buf needs to be a string file path or file-like')
+    else:
         return read(path_or_buf)
 
-    raise ValueError('path_or_buf needs to be a string file path or file-like')
-
 
 dtype_dict = {21: np.dtype('M8[ns]'),
               u('datetime64[ns]'): np.dtype('M8[ns]'),