Skip to content

Conversation

@HeartSaVioR
Copy link
Contributor

What changes were proposed in this pull request?

This PR adds toJVMRow in PythonSQLUtils to convert pickled PySpark Row to JVM Row.

Co-authored with @HyukjinKwon .

This is a breakdown PR of #37863.

Why are the changes needed?

This change will be leveraged in SPARK-40434.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

N/A. We will make sure test suites are constructed via E2E manner under SPARK-40431.

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. cc @ueshin FYI

@HeartSaVioR
Copy link
Contributor Author

HeartSaVioR commented Sep 15, 2022

Thanks! Merging to master.
(We have follow-up PRs so have enough chances to address the post review comments. Please feel free to leave review comments after merging this even minors.)

LuciferYang pushed a commit to LuciferYang/spark that referenced this pull request Sep 20, 2022
…ickled PySpark Row to JVM Row

### What changes were proposed in this pull request?

This PR adds toJVMRow in PythonSQLUtils to convert pickled PySpark Row to JVM Row.

Co-authored with HyukjinKwon .

This is a breakdown PR of apache#37863.

### Why are the changes needed?

This change will be leveraged in [SPARK-40434](https://issues.apache.org/jira/browse/SPARK-40434).

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

N/A. We will make sure test suites are constructed via E2E manner under [SPARK-40431](https://issues.apache.org/jira/browse/SPARK-40431).

Closes apache#37891 from HeartSaVioR/SPARK-40433.

Lead-authored-by: Jungtaek Lim <[email protected]>
Co-authored-by: Hyukjin Kwon <[email protected]>
Signed-off-by: Jungtaek Lim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants