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-1050: Update ORC site README.md and release process page #963

Merged
merged 1 commit into from
Nov 14, 2021
Merged

ORC-1050: Update ORC site README.md and release process page #963

merged 1 commit into from
Nov 14, 2021

Conversation

dongjoon-hyun
Copy link
Member

What changes were proposed in this pull request?

This PR aims to update ORC release and publish process pages.

Why are the changes needed?

The current web content is outdated.

How was this patch tested?

N/A

@github-actions github-actions bot added the DOCS label Nov 12, 2021
@dongjoon-hyun dongjoon-hyun marked this pull request as draft November 12, 2021 03:27
@dongjoon-hyun dongjoon-hyun changed the title ORC-1050: Update ORC release and publish process pages ORC-1050: Update ORC site README.md and release process page Nov 12, 2021
@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review November 12, 2021 03:36
@dongjoon-hyun
Copy link
Member Author

cc @williamhyun and @guiyanakuang

While reviewing @williamhyun 's PR, I realized that our website README and make-release page is outdated. This is the up-to-date process I used during Apache ORC 1.7.1 release.

@dongjoon-hyun dongjoon-hyun added this to the 1.8.0 milestone Nov 12, 2021
@dongjoon-hyun
Copy link
Member Author

Gentle ping, @williamhyun and @guiyanakuang .

@guiyanakuang
Copy link
Member

@dongjoon-hyun It looks good, but the url of the archive seems to be out of date in make-release.md, I think it can be updated together

Copy link
Member

@williamhyun williamhyun left a comment

Choose a reason for hiding this comment

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

+1 LGTM

@dongjoon-hyun
Copy link
Member Author

Thank you so much for review, @guiyanakuang and @williamhyun .

To @guiyanakuang . The archive link style is correct. I used the following for Apache ORC 1.7.1rc0. The link was just removed when I convert the release candidate tag, release-1.7.1rc0, to the official release tag, rel/release-1.7.1.

curl -LO https://github.com/apache/orc/archive/release-1.7.1rc0.tar.gz

@dongjoon-hyun dongjoon-hyun merged commit 22cb538 into apache:main Nov 14, 2021
@dongjoon-hyun
Copy link
Member Author

Merged to main for next release.

@dongjoon-hyun dongjoon-hyun deleted the ORC-1050 branch November 14, 2021 01:27
@guiyanakuang
Copy link
Member

I get it

dongjoon-hyun added a commit that referenced this pull request Feb 4, 2022
### What changes were proposed in this pull request?

This PR aims to update ORC release and publish process pages.

### Why are the changes needed?

The current web content is outdated.

### How was this patch tested?

N/A

(cherry picked from commit 22cb538)
Signed-off-by: Dongjoon Hyun <[email protected]>
@dongjoon-hyun dongjoon-hyun modified the milestones: 1.8.0, 1.7.3 Feb 4, 2022
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.

3 participants