search

scylladb / scylla-cqlsh

A fork of the cqlsh code
Apache License 2.0
16 stars 32 forks source link

cqlsh doesn't provide context sensitive help for service levels #43

Closed bhalevy closed 6 months ago

bhalevy commented 2 years ago

Maybe it doesn't also for other scylla-specific CQL features, but I noticed on 2021.1.6 that it's context sensitive tab-completion/help doesn't include e.g. LIST SERVICE_LEVEL; or LIST ALL SERVICE_LEVELS; or DROP SERVICE_LEVEL;

Cc @eliransin

eliransin commented 2 years ago

AFAICT this should be implemented in cqlsh itself. We probably should do it in our tool. Will try to do it when I've got time.

fruch commented 1 year ago

@mykaul can you move this one to http://github.com/scylladb/scylla-cqlsh

eliransin commented 1 year ago

@Jadw1 added this to your queue because it is service level related (no label for it in this current repo).

fruch commented 1 year ago

@eliransin @Jadw1 FYI I have a PR which address this: https://github.com/scylladb/scylla-cqlsh/pull/48

A review of it would be helpful, @Jadw1 you can that PR over if you want to make any changes to it.

Jadw1 commented 1 year ago

A review of it would be helpful, @Jadw1 you can that PR over if you want to make any changes to it.

I don't know cqlsh code very well so probably I don't have anything to add. But I'll look into the PR and check if it covers all of the cases (if i recall correctly you can write both LIST SERVICE_LEVEL and LIST SERVICE LEVEL)