-
Notifications
You must be signed in to change notification settings - Fork 483
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ORC-1185: Add merge_orc_pr.py
#1136
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Since this is a helper script which is irrelevant to ORC code itself, I'll merge this. |
1 similar comment
Since this is a helper script which is irrelevant to ORC code itself, I'll merge this. |
dongjoon-hyun
added a commit
that referenced
this pull request
Jul 7, 2022
### What changes were proposed in this pull request? This PR aims to add `merge_orc_pr.py` to help committers merge the PRs. ### Why are the changes needed? This is borrowed from [Apache Spark community](https://github.com/apache/spark/blob/master/dev/merge_spark_pr.py) which provides several features like - Supporting multi-author credits - Cleaning up the PR template comments - Landing on multiple branches - Handling Apache JIRA during merging process ### How was this patch tested? This should be tested manually after merging. Closes #1136 Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 7a37a03) Signed-off-by: Dongjoon Hyun <[email protected]>
I backported this to branch-1.8 as a part of Apache ORC 1.8.0 preparation. |
dongjoon-hyun
added a commit
that referenced
this pull request
Aug 5, 2022
### What changes were proposed in this pull request? This PR aims to add `merge_orc_pr.py` to help committers merge the PRs. ### Why are the changes needed? This is borrowed from [Apache Spark community](https://github.com/apache/spark/blob/master/dev/merge_spark_pr.py) which provides several features like - Supporting multi-author credits - Cleaning up the PR template comments - Landing on multiple branches - Handling Apache JIRA during merging process ### How was this patch tested? This should be tested manually after merging. Closes #1136 Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 7a37a03) Signed-off-by: Dongjoon Hyun <[email protected]>
Backported to branch-1.7. cc @williamhyun since he is the release manager of Apache ORC 1.7.6. |
dongjoon-hyun
added a commit
that referenced
this pull request
Jan 9, 2023
### What changes were proposed in this pull request? This PR is an accumulated change to make the PR labeler up-to-date. - Remove `.travis.yml` (#991) - Add `dev` folder (#1136) - Add `.clang-format` (#1308) ### Why are the changes needed? To make the PR labeler up-to-date. ### How was this patch tested? Manual review. Closes #1362 from dongjoon-hyun/ORC-1351. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
cxzl25
pushed a commit
to cxzl25/orc
that referenced
this pull request
Jan 11, 2024
### What changes were proposed in this pull request? This PR aims to add `merge_orc_pr.py` to help committers merge the PRs. ### Why are the changes needed? This is borrowed from [Apache Spark community](https://github.com/apache/spark/blob/master/dev/merge_spark_pr.py) which provides several features like - Supporting multi-author credits - Cleaning up the PR template comments - Landing on multiple branches - Handling Apache JIRA during merging process ### How was this patch tested? This should be tested manually after merging. Closes apache#1136 Signed-off-by: Dongjoon Hyun <[email protected]>
cxzl25
pushed a commit
to cxzl25/orc
that referenced
this pull request
Jan 11, 2024
### What changes were proposed in this pull request? This PR is an accumulated change to make the PR labeler up-to-date. - Remove `.travis.yml` (apache#991) - Add `dev` folder (apache#1136) - Add `.clang-format` (apache#1308) ### Why are the changes needed? To make the PR labeler up-to-date. ### How was this patch tested? Manual review. Closes apache#1362 from dongjoon-hyun/ORC-1351. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
This PR aims to add
merge_orc_pr.py
to help committers merge the PRs.Why are the changes needed?
This is borrowed from Apache Spark community which provides several features like
How was this patch tested?
This should be tested manually after merging.