Fix pyarrow Table type hinting in cursor.py (#1835)

* fix pyarrow Table type
* reorder imports within TYPE_CHECKING
* Use existing import check and improve warning
* remove import

Author: thomasaarholt

DESCRIPTION.md

@@ -14,6 +14,7 @@ Source code is also available at: https://github.com/snowflakedb/snowflake-conne
   - Cleanup some C++ code warnings and performance issues.
   - Added support for Python 3.12
   - Make local testing more robust against implicit assumptions.
+  - Fixed PyArrow Table type hinting
 
 - v3.6.0(December 09,2023)
 

src/snowflake/connector/cursor.py

@@ -72,6 +72,7 @@
 
 if TYPE_CHECKING:  # pragma: no cover
     from pandas import DataFrame
+    from pyarrow import Table
 
     from .connection import SnowflakeConnection
     from .file_transfer_agent import SnowflakeProgressPercentage
@@ -81,11 +82,14 @@
 
 logger = getLogger(__name__)
 
-if installed_pandas:
-    from pyarrow import Table
-else:
-    logger.debug("Failed to import pyarrow. Cannot use pandas fetch API")
-    Table = None
+
+if not installed_pandas:
+    logger.debug(
+        "Failed to import pyarrow or pandas. Cannot use pandas fetch API. Please "
+        "install snowflake-connector-python with the `pandas` extra to use these "
+        "features."
+    )
+
 
 try:
     from .nanoarrow_arrow_iterator import PyArrowIterator  # NOQA