Skip to content

Replace the deprecated api with new api in Spark 3.2#3490

Closed
zhaomin1423 wants to merge 1 commit intoapache:masterfrom
zhaomin1423:spark32
Closed

Replace the deprecated api with new api in Spark 3.2#3490
zhaomin1423 wants to merge 1 commit intoapache:masterfrom
zhaomin1423:spark32

Conversation

@zhaomin1423
Copy link
Member

Replace the deprecated api with new api in Spark 3.2.

@github-actions github-actions bot added the spark label Nov 7, 2021
@rdblue
Copy link
Contributor

rdblue commented Nov 7, 2021

#3461 implements a real Write to request distribution and ordering, so I think that is a better direction.

In addition, we generally don't want to make changes like this without a reason to. Cosmetic changes can cause commit conflicts and make it hard to maintain branches. It's okay to implement an older API if it means less code churn.

@rdblue rdblue closed this Nov 7, 2021
@zhaomin1423
Copy link
Member Author

#3461 implements a real Write to request distribution and ordering, so I think that is a better direction.

In addition, we generally don't want to make changes like this without a reason to. Cosmetic changes can cause commit conflicts and make it hard to maintain branches. It's okay to implement an older API if it means less code churn.

I see, Thanks!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants