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

Update ORC to 1.7.5 #12667

Merged
merged 1 commit into from
Jun 24, 2022
Merged

Update ORC to 1.7.5 #12667

merged 1 commit into from
Jun 24, 2022

Conversation

williamhyun
Copy link
Member

@williamhyun williamhyun commented Jun 17, 2022

Description

This PR aims to upgrade ORC to version 1.7.5.

ORC 1.7.5 is the latest version of ORC with the following patches:

This PR has:

  • been self-reviewed.
  • added or updated version, license, or notice information in licenses.yaml

Copy link
Member

@dongjoon-hyun dongjoon-hyun 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.

Copy link
Contributor

@cryptoe cryptoe left a comment

Choose a reason for hiding this comment

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

LGTM too!!

Copy link
Contributor

@gianm gianm left a comment

Choose a reason for hiding this comment

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

LGTM. thank you!

@gianm
Copy link
Contributor

gianm commented Jun 21, 2022

For some reason, Travis CI didn't run on this patch. I'll cycle it to get it to run.

@gianm gianm closed this Jun 21, 2022
@gianm gianm reopened this Jun 21, 2022
@gianm gianm closed this Jun 21, 2022
@gianm gianm reopened this Jun 21, 2022
@dongjoon-hyun
Copy link
Member

It looks identical. Is there another way to make CI happy?

@gianm
Copy link
Contributor

gianm commented Jun 24, 2022

Hmm, I'm not sure why it still won't run. This change seems low risk, though, so I'll merge it anyway. If it causes a problem for some reason we can revert.

@gianm gianm merged commit 2aadd69 into apache:master Jun 24, 2022
@dongjoon-hyun
Copy link
Member

Thank you so much, @gianm and @williamhyun ! :)

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.

5 participants