[ 
https://issues.apache.org/jira/browse/CASSANDRA-12946?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15688528#comment-15688528
 ] 

Stefania commented on CASSANDRA-12946:
--------------------------------------

Patch LGTM.

Running tests here:

||3.X||trunk||
|[patch|https://github.com/stef1927/cassandra/tree/12946-cqlsh-3.X]|[patch|https://github.com/stef1927/cassandra/tree/12946-cqlsh]|
|[tests|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-12946-cqlsh-3.X-cqlsh-tests/]|[tests|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-12946-cqlsh-cqlsh-tests/]|

> cqlsh auto completion: move definition of compaction strategy options to a 
> convenient place
> -------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-12946
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12946
>             Project: Cassandra
>          Issue Type: Task
>          Components: Tools
>            Reporter: Eduard Tudenhoefner
>            Assignee: Eduard Tudenhoefner
>              Labels: cqlsh
>             Fix For: 3.x
>
>
> For cqlsh auto completion the compaction strategy suboptions are currently 
> directly defined in 
> https://github.com/apache/cassandra/blob/cassandra-3.X/pylib/cqlshlib/cql3handling.py#L508-L527.
>  It would be more convenient to move them to *Cql3ParsingRuleSet* so that 
> they can be reused at other places.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to