Yikun commented on code in PR #36793:
URL: https://github.com/apache/spark/pull/36793#discussion_r894989727
##########
python/pyspark/sql/session.py:
##########
@@ -952,12 +953,29 @@ def createDataFrame( # type: ignore[misc]
schema = [x.encode("utf-8") if not isinstance(x, str) else x for x
in schema]
try:
- import pandas
+ import pandas as pd
has_pandas = True
except Exception:
has_pandas = False
- if has_pandas and isinstance(data, pandas.DataFrame):
+
+ try:
+ import numpy as np
+
+ has_numpy = True
+ except Exception:
+ has_numpy = False
+
+ if has_numpy and isinstance(data, np.ndarray):
+ from pyspark.sql.pandas.utils import require_minimum_pandas_version
+
+ require_minimum_pandas_version()
Review Comment:
nit: If only numpy installed but pandas not installed, will only raised
pandas not installed. User may confused: why need to install pandas but I just
want to using numpy ?
So maybe give a certainly exceptions before here to tell users, numpy type
will first convert pandas df in pyspark so pandas installed is required, like:
```python
if not has_pandas:
// warning or raised friendly exception
from pyspark.sql.pandas.utils import require_minimum_pandas_version
require_minimum_pandas_version()
```
or add a notes before here at least.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
