[
https://issues.apache.org/jira/browse/CASSANDRA-11719?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15286205#comment-15286205
]
Robert Stupp commented on CASSANDRA-11719:
------------------------------------------
Hm. You patch doesn't compile. I don't think that just calling
{{prepared.getBoundVariables().toString()}} is enough. You probably also need
the {{options.getValues()}} and format these values properly as strings using
{{type.asCQL3Type().toCQLLiteral()}} for each value.
Also, can you add some unit tests for this to {{TraceCqlTest}}?
Since this ticket is a new feature, it should go into trunk (an even 3.x
release).
> Add bind variables to trace
> ---------------------------
>
> Key: CASSANDRA-11719
> URL: https://issues.apache.org/jira/browse/CASSANDRA-11719
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Robert Stupp
> Assignee: Mahdi Mohammadi
> Priority: Minor
> Labels: lhf
> Fix For: 3.x
>
> Attachments: 11719-2.1.patch
>
>
> {{org.apache.cassandra.transport.messages.ExecuteMessage#execute}} mentions a
> _TODO_ saying "we don't have [typed] access to CQL bind variables here".
> In fact, we now have access typed access to CQL bind variables there. So, it
> is now possible to show the bind variables in the trace.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
