Skip to content

[SPARK-41801][CORE][PYTHON][PS] Remove *args: Any, **kwargs: Any for def transpose#39327

Closed
bjornjorgensen wants to merge 3 commits intoapache:masterfrom
bjornjorgensen:transpose
Closed

[SPARK-41801][CORE][PYTHON][PS] Remove *args: Any, **kwargs: Any for def transpose#39327
bjornjorgensen wants to merge 3 commits intoapache:masterfrom
bjornjorgensen:transpose

Conversation

@bjornjorgensen
Copy link
Contributor

What changes were proposed in this pull request?

Change def transpose(self, *args: Any, **kwargs: Any) -> "Series":
to
def transpose(self) -> "Series":

Why are the changes needed?

def transpose(self, *args: Any, **kwargs: Any) -> "Series":

This function only return self.copy()

Sonar

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA.

@bjornjorgensen bjornjorgensen changed the title [SPARK-41801][PS] Remove *args: Any, **kwargs: Any for def transpose [SPARK-41801][CORE][PYTHON][PS] Remove *args: Any, **kwargs: Any for def transpose Dec 31, 2022
agg = aggregate

def transpose(self, *args: Any, **kwargs: Any) -> "Series":
def transpose(self) -> "Series":
Copy link
Member

Choose a reason for hiding this comment

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

Hm, I think we should better keep it for now for pandas compatibility.

Copy link
Member

Choose a reason for hiding this comment

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

Or would be nicer if we implement these parameters

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@github-actions
Copy link

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Apr 13, 2023
@github-actions github-actions bot closed this Apr 14, 2023
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.

3 participants

Comments