4paradigm / 4paradigm/OpenMLDB
Improve CLI success info
- Dominant language
- C++
- Stars
- 1.7k
- Forks
- 331
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 1
Description
**Is your feature request related to a problem? Please describe.**
This issue is going to improve CLI output when SQL operate successfully.
For instance, when we query a create table statement in CLI successfully,
```sql
> CREATE TABLE abc (a INT);
```
The CLI will output text like:
```bash
SUCCEED: Create successfully
```
We expect a standard output like:
```
Query OK, 0 rows affected (0.09 sec)
```
which is far more familiar with SQL users.
**Describe the solution you'd like**
A clear and concise description of what you want to happen.
**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.
**Additional context**
Add any other context or screenshots about the feature request here.
Contributor guide
Research direction
Look for CLI output code, likely in a client or terminal interface module. Find where 'SUCCEED: Create successfully' is generated. Understand the SQL command execution flow to format a success message with rows affected and timing. Test changes by running CREATE TABLE statements in the CLI and verifying the new output matches the MySQL-style format.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- sql
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100