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

[Branch 2.7] upgrade aircompressor to 0.20 #16893

Merged
merged 2 commits into from Aug 1, 2022

Conversation

Jason918
Copy link
Contributor

@Jason918 Jason918 commented Aug 1, 2022

Motivation

Cherry-pick #11790 to branch 2.7

After directly cherry-pick #11790 , there is a license check issue:

aircompressor-0.16.jar unaccounted for in lib/presto/LICENSE

The root cause is that in branch 2.7, pulsar-presto-distribution is not a child of Pulsar main project. So the version changes in main pom.xml is not applied to lib/presto/LICENSE.

Modification

  1. Cherry-pick upgrade aircompressor to 0.20 #11790 to branch 2.7
  2. Add dependency of aircompressor-0.20 in pulsar-sql/presto-distribution/pom.xml.

Verifying this change

  • Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as CI license check.

Documentation

Check the box below or label this PR directly.

Need to update docs?

  • doc-not-needed
    bug fix

@Jason918
Copy link
Contributor Author

Jason918 commented Aug 1, 2022

/pulsarbot run-failure-checks

@Jason918 Jason918 merged commit c5aaac7 into apache:branch-2.7 Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants