zhengruifeng commented on code in PR #54180:
URL: https://github.com/apache/spark/pull/54180#discussion_r2803917622


##########
python/pyspark/sql/worker/create_data_source.py:
##########
@@ -125,6 +127,11 @@ def _main(infile: IO, outfile: IO) -> None:
                 # Here we cannot use _parse_datatype_string to parse the DDL 
string schema.
                 # as it requires an active Spark session.
                 is_ddl_string = True
+            if isinstance(schema, pa.schema):

Review Comment:
   I am wondering whether we can directly pass the pyarrow schema in this 
example
   
   ```
       class ArrowBatchDataSourceReader(DataSourceReader):
           def __init__(self, pa_schema, options):
               self.pa_schema: pa.Schema = pa_schema
   ```
   
   Not sure whether this works, @allisonwang-db is this allowed?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to