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

Remove unused dependency management from POM #6174

Closed
wants to merge 2 commits into from

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Mar 12, 2024

🔍 Description

Remove Thrift, Hive Service RPC, Derby dependency management from POM.

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Pass GA.


Checklist 📝

Be nice. Be informative.

@pan3793
Copy link
Member Author

pan3793 commented Mar 13, 2024

cc @zhouyifan279

@pan3793 pan3793 self-assigned this Mar 13, 2024
@pan3793 pan3793 added this to the v1.9.0 milestone Mar 13, 2024
@pan3793 pan3793 requested a review from ulysses-you March 13, 2024 03:15
@pan3793 pan3793 closed this in 3df2120 Mar 13, 2024
@pan3793
Copy link
Member Author

pan3793 commented Mar 13, 2024

Thanks, merged to master, and I checked the related GitHub CVE alerting has gone

zhaohehuhu pushed a commit to zhaohehuhu/incubator-kyuubi that referenced this pull request Mar 21, 2024
# 🔍 Description

Remove Thrift, Hive Service RPC, Derby dependency management from POM.

## Types of changes 🔖

- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

Pass GA.

---

# Checklist 📝

- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes apache#6174 from pan3793/rm-thrift.

Closes apache#6174

2f4cee6 [Cheng Pan] derby
46b6fdd [Cheng Pan] Remove thrift management from POM

Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
@pan3793 pan3793 deleted the rm-thrift branch June 6, 2024 18:41
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.

2 participants