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

Paulo Motta commented on CASSANDRA-10633:
-----------------------------------------

Code and CI looks good. Would you mind adding a simple regression dtest using 
the TIMEFORMAT option? 

Thanks!

> cqlsh copy uses wrong variable name for time_format
> ---------------------------------------------------
>
>                 Key: CASSANDRA-10633
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10633
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Tools
>            Reporter: Jeremiah Jordan
>            Assignee: Stefania
>             Fix For: 2.1.x, 2.2.x, 3.0.x
>
>
> {code}
> diff --git a/bin/cqlsh b/bin/cqlsh
> index ca45be3..ddf0314 100755
> --- a/bin/cqlsh
> +++ b/bin/cqlsh
> @@ -1816,7 +1816,7 @@ class Shell(cmd.Cmd):
>          encoding = opts.pop('encoding', 'utf8')
>          nullval = opts.pop('null', '')
>          header = bool(opts.pop('header', '').lower() == 'true')
> -        timestamp_format = opts.pop('time_format', 
> self.display_timestamp_format)
> +        timestamp_format = opts.pop('time_format', self.display_time_format)
>          if dialect_options['quotechar'] == dialect_options['escapechar']:
>              dialect_options['doublequote'] = True
>              del dialect_options['escape char']
> {code}



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

Reply via email to