Skip to content

[SPARK-58157][PYTHON] Remove TransformWithStateInPySparkRowInitStateSerializer#57438

Open
Yicong-Huang wants to merge 1 commit into
apache:masterfrom
Yicong-Huang:SPARK-58157
Open

[SPARK-58157][PYTHON] Remove TransformWithStateInPySparkRowInitStateSerializer#57438
Yicong-Huang wants to merge 1 commit into
apache:masterfrom
Yicong-Huang:SPARK-58157

Conversation

@Yicong-Huang

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Delete TransformWithStateInPySparkRowInitStateSerializer from python/pyspark/sql/pandas/serializers.py. Its init-state deserialization logic was moved into read_udfs() in worker.py by SPARK-58191 (#57337), which switched SQL_TRANSFORM_WITH_STATE_PYTHON_ROW_INIT_STATE_UDF to use the plain ArrowStreamSerializer for pure Arrow stream I/O. The class is no longer referenced by any eval type. A benchmark comment in bench_eval_type.py that attributed the row-by-row walk to this class is updated to point at read_udfs().

Part of 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants