-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
SHOW DATABASES shows one non-existing database #5185
Comments
"Name" is the name of the column. On Monday, December 21, 2015, Paulo Pires notifications@github.com wrote:
|
We have two |
I agree, I've never liked it either. It could do with improvement. |
@otoolep not sure how to follow from here. Change the formatting? Split |
I'd need to take a look at the code. Ideally we'd implement a general solution, that would add formattting to make it clear they are column names. |
@otoolep agreed. I have no idea if changing how the query responses are formatted would be a breaking change but am glad to own it if everyone agrees it's the correct step into fixing this. |
@pires as I clarified in another comment, the |
@beckettsean yes I had figured it out before but want you guys to (help) decide what the solution would be here. I provided two possible ways to fix it but the one me and @otoolep (and probably everyone) agree upon may be a breaking change, even if just an aesthetic one. |
Simply improving the output of the CLI such that the column names somehow are underlined would be fine a solution. It's not a breaking change since the CLI output is not part of the API. E.g.:
|
Then again, perhaps this is not that clear either. :-( But you get my point. |
@otoolep I like that suggestion. I think it improves readability across the board. |
It's still a little funny looking that
Closing this issue. |
name
is a non-existing database.Refs #5183
The text was updated successfully, but these errors were encountered: