Skip to content

Commit

Permalink
complete tests
Browse files Browse the repository at this point in the history
  • Loading branch information
xinrong-meng committed Nov 13, 2024
1 parent e5af00a commit 2f7c30d
Showing 1 changed file with 18 additions and 2 deletions.
20 changes: 18 additions & 2 deletions python/pyspark/sql/tests/test_arrow.py
Original file line number Diff line number Diff line change
Expand Up @@ -534,7 +534,7 @@ def test_createDataFrame_arrow_pandas(self):

def test_createDataFrame_verifySchema(self):
data = {"id": [1, 2, 3], "value": [100000000000, 200000000000, 300000000000]}
table = pa.table(data)
# data.value should fail schema validation when verifySchema is True
schema = StructType(
[StructField("id", IntegerType(), True), StructField("value", IntegerType(), True)]
)
Expand All @@ -543,13 +543,29 @@ def test_createDataFrame_verifySchema(self):
Row(id=2, value=-1863462912),
Row(id=3, value=-647710720),
]

# Arrow table
table = pa.table(data)
df = self.spark.createDataFrame(table, schema=schema)
self.assertEqual(df.collect(), expected)

with self.assertRaises(Exception):
self.spark.createDataFrame(table, schema=schema, verifySchema=True)

# pandas DataFrame with Arrow optimization
pdf = pd.DataFrame(data)
df = self.spark.createDataFrame(pdf, schema=schema) # verifySchema defaults to False
self.assertEqual(df.collect(), expected)
with self.assertRaises(Exception):
df = self.spark.createDataFrame(pdf, schema=schema, verifySchema=True)

# pandas DataFrame without Arrow optimization
with self.sql_conf({"spark.sql.execution.arrow.pyspark.enabled": False}):
pdf = pd.DataFrame(data)
with self.assertRaises(Exception):
df = self.spark.createDataFrame(pdf, schema=schema) # verifySchema defaults to True
df = self.spark.createDataFrame(pdf, schema=schema, verifySchema=False)
self.assertEqual(df.collect(), expected)

def _createDataFrame_toggle(self, data, schema=None):
with self.sql_conf({"spark.sql.execution.arrow.pyspark.enabled": False}):
df_no_arrow = self.spark.createDataFrame(data, schema=schema)
Expand Down

0 comments on commit 2f7c30d

Please sign in to comment.