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.

Sign up for GitHub

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

Jump to bottom

Desc not existed user should report error #5141

Closed
Shylock-Hg opened this issue Dec 28, 2022 · 3 comments · Fixed by #5345
Closed

Desc not existed user should report error #5141

Shylock-Hg opened this issue Dec 28, 2022 · 3 comments · Fixed by #5345
Labels
affects/master PR/issue: this bug affects master version. process/fixed Process of bug severity/minor Severity of bug type/bug Type: something is unexpected
Milestone
v3.5.0

Comments

@Shylock-Hg
Copy link
Contributor

Please check the FAQ documentation before raising an issue

Describe the bug (required)

Found in #5139

Your Environments (required)

  • OS: uname -a
  • Compiler: g++ --version or clang++ --version
  • CPU: lscpu
  • Commit id (e.g. a3ffc7d8)

How To Reproduce(required)

Steps to reproduce the behavior:

  1. Step 1
  2. Step 2
  3. Step 3

Expected behavior

Additional context
@Shylock-Hg Shylock-Hg added the type/bug Type: something is unexpected label Dec 28, 2022
@github-actions github-actions bot added affects/none PR/issue: this bug affects none version. severity/none Severity of bug labels Dec 28, 2022
@Sophie-Xie Sophie-Xie added this to the v3.4.0 milestone Dec 28, 2022
@Sophie-Xie Sophie-Xie added severity/minor Severity of bug and removed severity/none Severity of bug labels Dec 28, 2022
@wey-gu wey-gu mentioned this issue Dec 31, 2022
Closed
@HarrisChu
Copy link
Contributor

what's the current behavior?

@Shylock-Hg
Copy link
Contributor Author

what's the current behavior?

Empty result.

@HarrisChu HarrisChu added the affects/master PR/issue: this bug affects master version. label Jan 3, 2023
@github-actions github-actions bot removed the affects/none PR/issue: this bug affects none version. label Jan 3, 2023
@Shylock-Hg Shylock-Hg removed their assignment Jan 4, 2023
@Sophie-Xie Sophie-Xie modified the milestones: v3.4.0, v3.5.0 Jan 4, 2023
@Milittle
Copy link
Contributor

Can i try this? thx

@Milittle Milittle mentioned this issue Feb 15, 2023
Merged
11 tasks
@github-actions github-actions bot added the process/fixed Process of bug label Feb 20, 2023
@wey-gu wey-gu mentioned this issue Feb 25, 2023
Closed
yixinglu pushed a commit to yixinglu/nebula that referenced this issue Sep 14, 2023
@nebula-bots @Milittle
fix if user does not exist, desc user xxx will be raise "User not e…
2c8d0ff 
…xisted!" (vesoft-inc#2450)

If user does not exist `desc user xxx` will be raise "User not existed!"

<!--
Thanks for your contribution!
In order to review PR more efficiently, please add information according to the template.
-->

## What type of PR is this?
- [x] bug
- [ ] feature
- [ ] enhancement

## What problem(s) does this PR solve?
#### Issue(s) number: 
Close vesoft-inc#5141

#### Description:
If use does not exist `desc user xxx` will be raise "User not existed!"

## How do you solve it?
add account exist verify in meta service

## Special notes for your reviewer, ex. impact of this fix, design document, etc:

## Checklist:
Tests:
manunal test:
```
(root@nebula) [(none)]> desc user u6
[ERROR (-1005)]: User not existed!

Wed, 15 Feb 2023 22:43:06 CST
```
- [ ] Unit test(positive and negative cases)
- [ ] Function test
- [ ] Performance test
- [ ] N/A

Affects:
- [ ] Documentation affected (Please add the label if documentation needs to be modified.)
- [ ] Incompatibility (If it breaks the compatibility, please describe it and add the label.）
- [ ] If it's needed to cherry-pick (If cherry-pick to some branches is required, please label the destination version(s).)
- [ ] Performance impacted: Consumes more CPU/Memory


## Release notes:

Please confirm whether to be reflected in release notes and how to describe:
> ex. Fixed the bug .....


Migrated from vesoft-inc#5345

Co-authored-by: Milittle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects/master PR/issue: this bug affects master version. process/fixed Process of bug severity/minor Severity of bug type/bug Type: something is unexpected
Projects
None yet
Milestone
v3.5.0
Development

Successfully merging a pull request may close this issue.

If user does not exist desc user xxx will be raise "User not existed!" Milittle/nebula
4 participants
@HarrisChu @Milittle @Shylock-Hg @Sophie-Xie