Skip to content
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
wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented May 23, 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 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.

@dongjoon-hyun dongjoon-hyun added this to the 1.9.0 milestone May 23, 2022
@dongjoon-hyun
Copy link
Member Author

Since this is a helper script which is irrelevant to ORC code itself, I'll merge this.

1 similar comment
@dongjoon-hyun
Copy link
Member Author

Since this is a helper script which is irrelevant to ORC code itself, I'll merge this.

@dongjoon-hyun dongjoon-hyun deleted the ORC-1185 branch May 24, 2022 05:06
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]>
@dongjoon-hyun dongjoon-hyun modified the milestones: 1.9.0, 1.8.0 Jul 7, 2022
@dongjoon-hyun
Copy link
Member Author

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]>
@dongjoon-hyun dongjoon-hyun modified the milestones: 1.8.0, 1.7.6 Aug 5, 2022
@dongjoon-hyun
Copy link
Member Author

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant