-
Notifications
You must be signed in to change notification settings - Fork 1k
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
ksqldb CLI should use ksqlDB prompt #4071
Comments
Are you saying we don't want to change the prompt? Or we're just saying that it's ok as it is for the moment with the reasoning given
|
the two extra |
@rmoff we don’t plan to change the prompt. The issue is that it becomes a slippery slope: do we also change config parameter names? Do we change every occurrence of “ksql”? And would the resulting disruption be worth it to users? As @agavra alluded to, it’s very common for a system like this to have a long and short name. e.g. Mongo/MongoDB, Postgres/PostgreSQL, etc., and in our case I don’t think that doing this meaningfully detracts from the UX. |
Let's please not do this "just because". I seem to recall that using
sql/plus still gives me a "sql>" prompt rather than one that says
"Oracle:>" ;)
Other systems take a variety of approaches, there's no overwhelming
convention I can see. (Personal favorite is the way SQLServer sets the
prompt to be the name of the schema you are in but we don't have that
concept yet)
…On Fri, Dec 6, 2019, 9:27 AM Derek Nelson ***@***.***> wrote:
@rmoff <https://github.com/rmoff> we don’t plan to change the prompt. The
issue is that it becomes a slippery slope: do we also change config
parameter names? Do we change every occurrence of “ksql”? And would the
resulting disruption be worth it to users?
As @agavra <https://github.com/agavra> alluded to, it’s very common for a
system like this to have a long and short name. e.g. Mongo/MongoDB,
Postgres/PostgreSQL, etc., and in our case I don’t think that doing this
meaningfully detracts from the UX.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#4071?email_source=notifications&email_token=ABCXJIGJJFHNRTSXJTEHFPDQXKDP7A5CNFSM4JW22B52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGEZG4Q#issuecomment-562664306>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCXJIEWFRHNNRYCLW6UGX3QXKDP7ANCNFSM4JW22B5Q>
.
|
This is a great way to use the prompt and is what Postgres does as well (prompt is the name of the database you're connected to). Introducing a schema/namespace abstraction is on our radar: #3966. |
ksqlDB 0.6
The
ksql>
prompt should beksqlDB>
The text was updated successfully, but these errors were encountered: