Yicong-Huang opened a new pull request, #57470:
URL: https://github.com/apache/spark/pull/57470

   ### What changes were proposed in this pull request?
   
   Delete `TransformWithStateInPySparkRowSerializer` from 
`python/pyspark/sql/pandas/serializers.py`. Its row-by-row deserialization and 
re-encoding logic was moved into `read_udfs()` in `worker.py` by 
[SPARK-58191](https://issues.apache.org/jira/browse/SPARK-58191) (#57337), 
which switched `SQL_TRANSFORM_WITH_STATE_PYTHON_ROW_UDF` to use the plain 
`ArrowStreamSerializer` for pure Arrow stream I/O. The class is no longer 
referenced by any eval type. The now-unused `itertools.groupby` and 
`pyspark.sql.Row` imports it required are also removed, and two benchmark 
comments in `bench_eval_type.py` that attributed the row walk to this class are 
updated to point at the worker.
   
   Part of [SPARK-55384](https://issues.apache.org/jira/browse/SPARK-55384).
   
   ### Why are the changes needed?
   
   The class is dead code after the refactor in SPARK-58191. Removing it keeps 
serializers as pure I/O and avoids a stale, unused class.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Existing tests. This is a dead-code deletion with no remaining references 
(verified repo-wide, including dynamic lookups); behavior is unchanged since 
the eval type already runs through `read_udfs()`.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No.
   


-- 
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