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

[TASK][TRIVIAL] Fix the usage msg of KyuubiBeeLine #5822

Closed
3 tasks done
hadoopkandy opened this issue Dec 6, 2023 · 0 comments
Closed
3 tasks done

[TASK][TRIVIAL] Fix the usage msg of KyuubiBeeLine #5822

hadoopkandy opened this issue Dec 6, 2023 · 0 comments
Assignees

Comments

@hadoopkandy
Copy link
Contributor

hadoopkandy commented Dec 6, 2023

What's the level of this task?

TRIVIAL

Code of Conduct

Search before creating

  • I have searched in the task list and found no similar tasks.

Mentor

  • I have sufficient expertise on this task, and I volunteer to be a mentor of this task to guide contributors through the task.

Skill requirements

Familiar With KyuubiBeeLine

Background and Goals

Familiar With KyuubiBeeLine

Implementation steps

Fix the usage msg of KyuubiBeeLine

Additional context

Introduction of 2023 Kyuubi Code Contribution Program

@hadoopkandy hadoopkandy changed the title [TASK][<EASY>] [doc] Fix the usage msg of KyuubiBeeLine [TASK][EASY] [doc] Fix the usage msg of KyuubiBeeLine Dec 6, 2023
hadoopkandy added a commit to hadoopkandy/kyuubi that referenced this issue Dec 6, 2023
@pan3793 pan3793 changed the title [TASK][EASY] [doc] Fix the usage msg of KyuubiBeeLine [TASK][TRIVIAL] Fix the usage msg of KyuubiBeeLine Dec 7, 2023
@pan3793 pan3793 closed this as completed in 5b3a78d Dec 7, 2023
pan3793 pushed a commit that referenced this issue Dec 7, 2023
# 🔍 Description
## Issue References 🔗

This pull request fixes #5822

## Describe Your Solution 🔧

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

## 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 ⚰️

```
$ $KYUUBI_HOME/bin/beeline --help
...
Usage: java " + KyuubiBeeLine.class.getCanonicalName()
...
```

#### Behavior With This Pull Request 🎉

```
$ $KYUUBI_HOME/bin/beeline --help
...
Usage: java org.apache.hive.beeline.KyuubiBeeLine
...
```

#### Related Unit Tests

---

# Checklists
## 📝 Author Self Checklist

- [x] My code follows the [style guidelines](https://kyuubi.readthedocs.io/en/master/contributing/code/style.html) of this project
- [x] I have performed a self-review
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

## 📝 Committer Pre-Merge Checklist

- [x] Pull request title is okay.
- [x] No license issues.
- [x] Milestone correctly set?
- [ ] Test coverage is ok
- [x] Assignees are selected.
- [x] Minimum number of approvals
- [x] No changes are requested

**Be nice. Be informative.**

Closes #5823 from hadoopkandy/KYUUBI-5822.

Closes #5822

a46b6e4 [kandy01.wang] [KYUUBI #5822][doc] Fix the usage msg of KyuubiBeeLine

Authored-by: kandy01.wang <kandy01.wang@vipshop.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit 5b3a78d)
Signed-off-by: Cheng Pan <chengpan@apache.org>
pan3793 pushed a commit that referenced this issue Dec 7, 2023
# 🔍 Description
## Issue References 🔗

This pull request fixes #5822

## Describe Your Solution 🔧

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

## 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 ⚰️

```
$ $KYUUBI_HOME/bin/beeline --help
...
Usage: java " + KyuubiBeeLine.class.getCanonicalName()
...
```

#### Behavior With This Pull Request 🎉

```
$ $KYUUBI_HOME/bin/beeline --help
...
Usage: java org.apache.hive.beeline.KyuubiBeeLine
...
```

#### Related Unit Tests

---

# Checklists
## 📝 Author Self Checklist

- [x] My code follows the [style guidelines](https://kyuubi.readthedocs.io/en/master/contributing/code/style.html) of this project
- [x] I have performed a self-review
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

## 📝 Committer Pre-Merge Checklist

- [x] Pull request title is okay.
- [x] No license issues.
- [x] Milestone correctly set?
- [ ] Test coverage is ok
- [x] Assignees are selected.
- [x] Minimum number of approvals
- [x] No changes are requested

**Be nice. Be informative.**

Closes #5823 from hadoopkandy/KYUUBI-5822.

Closes #5822

a46b6e4 [kandy01.wang] [KYUUBI #5822][doc] Fix the usage msg of KyuubiBeeLine

Authored-by: kandy01.wang <kandy01.wang@vipshop.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit 5b3a78d)
Signed-off-by: Cheng Pan <chengpan@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant