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

[Improvement] There's no colums field name in gcli output, it's confusing what's the null value meaning #5333

Closed
danhuawang opened this issue Oct 29, 2024 · 1 comment · Fixed by #5350
Closed

[Improvement] There's no colums field name in gcli output, it's confusing what's the null value meaning #5333

danhuawang opened this issue Oct 29, 2024 · 1 comment · Fixed by #5350
Assignees
@justinmclean
Labels
0.8.0 Release v0.8.0 improvement Improvements on everything

Comments

@danhuawang
Copy link
Contributor

What would you like to be improved?

image

How should we improve?

Print field name in the first line like:
name type comment nullable
@danhuawang danhuawang added the improvement Improvements on everything label Oct 29, 2024
@danhuawang
Copy link
Contributor Author

@justinmclean Can you help check this issue ?

@danhuawang danhuawang changed the title [Improvement] There's no field name in gcli output, it's confusing what's the null value meaning [Improvement] There's no colums' field name in gcli output, it's confusing what's the null value meaning Oct 29, 2024
@danhuawang danhuawang changed the title [Improvement] There's no colums' field name in gcli output, it's confusing what's the null value meaning [Improvement] There's no colums field name in gcli output, it's confusing what's the null value meaning Oct 29, 2024
@justinmclean justinmclean self-assigned this Oct 29, 2024
@justinmclean justinmclean mentioned this issue Oct 29, 2024
Merged
jerryshao pushed a commit that referenced this issue Nov 8, 2024
@justinmclean
[#5333] Add column title row to table column output in Gravitino CLI (#…
f3fed4a 
…5350)

### What changes were proposed in this pull request?

Add column title row to table column output.

### Why are the changes needed?

So it's easier for a user to understand the output.

Fix: #5333

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Compiled and tested locally.
@jerryshao jerryshao added the 0.8.0 Release v0.8.0 label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@justinmclean justinmclean

Labels
0.8.0 Release v0.8.0 improvement Improvements on everything
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#5333] Add column title row to table column output in Gravitino CLI justinmclean/gravitino
3 participants
@justinmclean @jerryshao @danhuawang