Skip to content

Conversation

@devin-petersohn
Copy link

What changes were proposed in this pull request?

New parameter axis=1 support for pandas.DataFrame.any

Why are the changes needed?

To introduce axis=1

Does this PR introduce any user-facing change?

Yes, any will now support axis=1.

How was this patch tested?

Locally, new tests

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

No

Signed-off-by: Devin Petersohn <[email protected]>
Signed-off-by: Devin Petersohn <[email protected]>
Copy link
Contributor

@holdenk holdenk left a comment

Choose a reason for hiding this comment

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

Thanks for working on this! Quick question regarding the validate change :) CC @HyukjinKwon who mentioned being interested in reviewing this when we chatted :)

@HyukjinKwon HyukjinKwon changed the title [SPARK-46166][Pyspark] Implementation of pandas.DataFrame.any with axis=1 [SPARK-46166][PS] Implementation of pandas.DataFrame.any with axis=1 Nov 18, 2025
@devin-petersohn
Copy link
Author

I believe that all comments so far have been addressed. Any other changes needed to get this across the finish line?

@HyukjinKwon
Copy link
Member

LGTM let me leave it to @holdenk to merge

@holdenk
Copy link
Contributor

holdenk commented Nov 20, 2025

LGTM I'll merge it this week (currently on a work trip) :)

Copy link
Contributor

@holdenk holdenk left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@huaxingao huaxingao left a comment

Choose a reason for hiding this comment

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

LGTM

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.

6 participants