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

[KYUUBI #6145][FOLLOWUP] Remove unused dependency commons-lang #6161

Closed
wants to merge 1 commit into from

Conversation

zhouyifan279
Copy link
Contributor

@zhouyifan279 zhouyifan279 commented Mar 11, 2024

🔍 Description

Issue References 🔗

This pull request removes unused dependency commons-lang from kyuubi-server pom.xml.

Describe Your Solution 🔧

Removes unused dependency commons-lang from kyuubi-server pom.xml.

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 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests


Checklist 📝

Be nice. Be informative.

@pan3793 pan3793 added this to the v1.9.0 milestone Mar 11, 2024
@pan3793 pan3793 closed this in bb54334 Mar 11, 2024
@pan3793
Copy link
Member

pan3793 commented Mar 11, 2024

Thanks, merged to master

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

This pull request removes unused dependency commons-lang from kyuubi-server pom.xml.

## Describe Your Solution 🔧

Removes unused dependency commons-lang from kyuubi-server pom.xml.

## 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 🧪

#### Behavior Without This Pull Request ⚰️

#### Behavior With This Pull Request 🎉

#### Related Unit Tests

---

# 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#6161 from zhouyifan279/commons-lang.

Closes apache#6145

8d50b66 [zhouyifan279] [KYUUBI apache#6145][FOLLOWUP] Remove unused dependency commons-lang

Authored-by: zhouyifan279 <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
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