Revert "simplify imports"

This reverts commit 6919a70.

Author: jreback

pandas/io/orc.py

@@ -1,5 +1,6 @@
 """ orc compat """
 
+import distutils
 from typing import List, Optional
 
 from pandas.compat._optional import import_optional_dependency
@@ -11,13 +12,28 @@
 
 
 def get_engine(engine: str) -> "PyArrowImpl":
-    """ return our implementation; we only support a pyarrow impl """
+    """ return our implementation """
 
     if engine == "auto":
         engine = get_option("io.orc.engine")
 
+    if engine == "auto":
+        # try engines in this order
+        try:
+            return PyArrowImpl()
+        except ImportError:
+            pass
+
+        raise ImportError(
+            "Unable to find a usable engine; "
+            "tried using: 'pyarrow'.\n"
+            "pyarrow is required for orc "
+            "support"
+        )
+
     if engine not in ["pyarrow"]:
         raise ValueError("engine must be 'pyarrow'")
+
     return PyArrowImpl()
 
 
@@ -27,15 +43,12 @@ def __init__(self):
             "pyarrow", extra="pyarrow is required for orc support."
         )
 
-        try:
-            import pyarrow
-        except ImportError:
-            raise ImportError(
-                "Unable to find a usable engine; "
-                "tried using: 'pyarrow'.\n"
-                "pyarrow is required for orc "
-                "support"
-            )
+        # we require a newer version of pyarrow thaN we support for parquet
+        import pyarrow
+
+        if distutils.version.LooseVersion(pyarrow.__version__) < "0.13.0":
+            raise ImportError("pyarrow must be >= 0.13.0 for read_orc")
+
         import pyarrow.orc
 
         self.api = pyarrow
@@ -44,7 +57,9 @@ def read(
         self, path: FilePathOrBuffer, columns: Optional[List[str]] = None, **kwargs
     ) -> DataFrame:
         path, _, _, _ = get_filepath_or_buffer(path)
-        orc_file = self.api.orc.ORCFile(path)
+
+        py_file = self.api.input_stream(path)
+        orc_file = self.api.orc.ORCFile(py_file)
 
         result = orc_file.read(columns=columns, **kwargs).to_pandas()
 

pandas/tests/io/test_orc.py

@@ -1,17 +1,27 @@
 """ test orc compat """
 import datetime
-from decimal import Decimal
+import distutils
 import os
 
 import numpy as np
 import pytest
 
+from pandas.compat import is_platform_windows
+
 import pandas as pd
 import pandas.util.testing as tm
 
 from pandas.io.orc import PyArrowImpl, get_engine, read_orc
 
-pyarrow = pytest.importorskip("pyarrow")
+try:
+    import pyarrow  # noqa
+
+    if distutils.version.LooseVersion(pyarrow.__version__) < "0.13.0":
+        raise ImportError("pyarrow must be >= 0.13.0 for read_orc")
+
+    _HAVE_PYARROW = True
+except ImportError:
+    _HAVE_PYARROW = False
 
 pytestmark = pytest.mark.filterwarnings(
     "ignore:RangeIndex.* is deprecated:DeprecationWarning"
@@ -23,7 +33,16 @@ def dirpath(datapath):
     return datapath("io", "data", "orc")
 
 
-def test_options_get_engine():
+@pytest.fixture
+def pa():
+    if not _HAVE_PYARROW:
+        pytest.skip("pyarrow is not installed")
+    if is_platform_windows():
+        pytest.skip("pyarrow orc not available by default on windows")
+    return "pyarrow"
+
+
+def test_options_get_engine(pa):
     assert isinstance(get_engine("pyarrow"), PyArrowImpl)
 
     with pd.option_context("io.orc.engine", "pyarrow"):
@@ -42,7 +61,7 @@ def test_invalid_engine(dirpath):
         read_orc(inputfile, engine=engine, columns=["boolean1"])
 
 
-def test_orc_reader_empty(dirpath):
+def test_orc_reader_empty(dirpath, pa):
     columns = [
         "boolean1",
         "byte1",
@@ -75,7 +94,7 @@ def test_orc_reader_empty(dirpath):
     tm.assert_equal(expected, got)
 
 
-def test_orc_reader_basic(dirpath):
+def test_orc_reader_basic(dirpath, pa):
     data = {
         "boolean1": np.array([False, True], dtype="bool"),
         "byte1": np.array([1, 100], dtype="int8"),
@@ -95,7 +114,8 @@ def test_orc_reader_basic(dirpath):
     tm.assert_equal(expected, got)
 
 
-def test_orc_reader_decimal(dirpath):
+def test_orc_reader_decimal(dirpath, pa):
+    from decimal import Decimal
 
     # Only testing the first 10 rows of data
     data = {
@@ -123,7 +143,7 @@ def test_orc_reader_decimal(dirpath):
     tm.assert_equal(expected, got)
 
 
-def test_orc_reader_date_low(dirpath):
+def test_orc_reader_date_low(dirpath, pa):
     data = {
         "time": np.array(
             [
@@ -164,7 +184,7 @@ def test_orc_reader_date_low(dirpath):
     tm.assert_equal(expected, got)
 
 
-def test_orc_reader_date_high(dirpath):
+def test_orc_reader_date_high(dirpath, pa):
     data = {
         "time": np.array(
             [
@@ -205,7 +225,7 @@ def test_orc_reader_date_high(dirpath):
     tm.assert_equal(expected, got)
 
 
-def test_orc_reader_snappy_compressed(dirpath):
+def test_orc_reader_snappy_compressed(dirpath, pa):
     data = {
         "int1": np.array(
             [