4paradigm / 4paradigm/OpenMLDB

Improve CLI success info

Open
#849 0 comments 0 reactions 1 assignee Claimed by @jingchen2222 View on GitHub
enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.