Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[SPARK-59455][PYTHON] Move Arrow-to-pandas batch conversion into ArrowToPandasConversion - #58754

Open
Spenserrrr wants to merge 1 commit into
apache:masterfrom
Spenserrrr:arrow-to-pandas-refactor
Open

[SPARK-59455][PYTHON] Move Arrow-to-pandas batch conversion into ArrowToPandasConversion#58754
Spenserrrr wants to merge 1 commit into
apache:masterfrom
Spenserrrr:arrow-to-pandas-refactor

Conversation

@Spenserrrr

@Spenserrrr Spenserrrr commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Rename ArrowArrayToPandasConversion to ArrowToPandasConversion and move ArrowBatchTransformer.to_pandas into it, updating callers, tests, and benchmarks.

Why are the changes needed?

#53963 has a review discussion about moving to_pandas into ArrowToPandasConversion. This groups batch and column conversion in the directional converter and keeps ArrowBatchTransformer focused on Arrow-to-Arrow operations.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing conversion tests and baseline-versus-refactor parity probes under pandas 2 and 3, plus benchmark smoke checks.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Codex (GPT-6)

@Spenserrrr Spenserrrr changed the title [SPARK-59454][PYTHON] Move Arrow-to-pandas batch conversion into ArrowToPandasConversion [SPARK-59455][PYTHON] Move Arrow-to-pandas batch conversion into ArrowToPandasConversion Sep 12, 2026
@Spenserrrr
Spenserrrr marked this pull request as ready for review September 13, 2026 07:45
@Spenserrrr

Copy link
Copy Markdown
Contributor Author

Hi @Yicong-Huang @zhengruifeng! This PR moves to_pandas into ArrowToPandasConversion, following the discussion in #53963. Could you take a look when you have time? Thanks!
cc @gaogaotiantian

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.

1 participant