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-1080: Remove .travis.yml #997

Merged
merged 1 commit into from
Jan 7, 2022
Merged

ORC-1080: Remove .travis.yml #997

merged 1 commit into from
Jan 7, 2022

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Jan 7, 2022

What changes were proposed in this pull request?

This PR aims to remove Travis CI from Apache ORC CIs.

Why are the changes needed?

In these days, Travis CI takes 4 or more hours to run.

ORC-1079 already provides the same test coverage in GitHub Action. So, we can remove Travis CI. safely.

How was this patch tested?

Travis CI should not be invoked in this PR.

@github-actions github-actions bot added the INFRA label Jan 7, 2022
@dongjoon-hyun
Copy link
Member Author

cc @guiyanakuang and @williamhyun

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

@williamhyun williamhyun merged commit bfd8fde into apache:main Jan 7, 2022
@dongjoon-hyun
Copy link
Member Author

Thank you, @williamhyun !

@dongjoon-hyun dongjoon-hyun deleted the ORC-1080 branch January 7, 2022 06:17
dongjoon-hyun added a commit that referenced this pull request Jan 7, 2022
### What changes were proposed in this pull request?

This PR aims to remove Travis CI from Apache ORC CIs.

### Why are the changes needed?

In these days, Travis CI takes 4 or more hours to run.
![](https://user-images.githubusercontent.com/9700541/148463375-beef2e9b-61c0-4fb1-bc8f-10b53b854b30.png)

ORC-1079 already provides the same test coverage in GitHub Action. So, we can remove `Travis CI`. safely.

### How was this patch tested?

Travis CI should not be invoked in this PR.

(cherry picked from commit bfd8fde)
Signed-off-by: Dongjoon Hyun <[email protected]>
@dongjoon-hyun dongjoon-hyun added this to the 1.7.3 milestone Jan 7, 2022
@dongjoon-hyun
Copy link
Member Author

I backported this to branch-1.7.

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

This PR aims to remove Travis CI from Apache ORC CIs.

### Why are the changes needed?

In these days, Travis CI takes 4 or more hours to run.
![](https://user-images.githubusercontent.com/9700541/148463375-beef2e9b-61c0-4fb1-bc8f-10b53b854b30.png)

ORC-1079 already provides the same test coverage in GitHub Action. So, we can remove `Travis CI`. safely.

### How was this patch tested?

Travis CI should not be invoked in this PR.
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 remove Travis CI from Apache ORC CIs.

### Why are the changes needed?

In these days, Travis CI takes 4 or more hours to run.
![](https://user-images.githubusercontent.com/9700541/148463375-beef2e9b-61c0-4fb1-bc8f-10b53b854b30.png)

ORC-1079 already provides the same test coverage in GitHub Action. So, we can remove `Travis CI`. safely.

### How was this patch tested?

Travis CI should not be invoked in this PR.
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