Skip to content

Conversation

@felixwluo
Copy link
Contributor

@felixwluo felixwluo commented Jul 2, 2024

For the alter operation, add partitions in batches
doris pr: apache/doris#37114

@shasevendow
Copy link
Contributor

Please attach the code PR link.

@felixwluo
Copy link
Contributor Author

Please attach the code PR link.

done

Copy link
Contributor

@shasevendow shasevendow left a comment

Choose a reason for hiding this comment

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

LGTM

@shasevendow
Copy link
Contributor

Does the 2.1 version need to be updated too?

@felixwluo
Copy link
Contributor Author

Does the 2.1 version need to be updated too?

Yes, this feature is implemented in 2.1

@shasevendow
Copy link
Contributor

Does the 2.1 version need to be updated too?

Yes, this feature is implemented in 2.1

Please refer to the contribution guide to update the 2.1 version docs: https://doris.apache.org/zh-CN/community/how-to-contribute/contribute-doc/

Copy link
Contributor

@KassieZ KassieZ left a comment

Choose a reason for hiding this comment

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

please also update version 3.0 / 2.1 / 2.0

@felixwluo felixwluo force-pushed the add-alter-partition-doc branch from 1332c5d to eff8ad1 Compare August 1, 2024 06:14
@felixwluo
Copy link
Contributor Author

please also update version 3.0 / 2.1 / 2.0

done

@KassieZ
Copy link
Contributor

KassieZ commented Sep 2, 2024

Hi, thank you for your contribution. The build cannot be triggered, so it is recommended to pull request again.

@KassieZ KassieZ added bug Something is incorrect and removed Reviewed labels Sep 2, 2024
@felixwluo
Copy link
Contributor Author

felixwluo commented Sep 2, 2024 via email

@felixwluo
Copy link
Contributor Author

Hi, thank you for your contribution. The build cannot be triggered, so it is recommended to pull request again.

#1077

@felixwluo felixwluo closed this Sep 3, 2024
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