pt-query-digest parses C-style comments incorrectly

[Luis Motta Campos]

This reports concerns pt-query-digest 2.1.1.
This is the command line I'm using: 'pt-query-digest --report slow.log'
The version of the server producing the slow query log: 5.1.56-rel12.7-log ((Percona Server (GPL), 12.7, Revision 237))

[Luis Motta Campos (luismottacampos)]

Sorry, there's no way to edit a bug description and I've pressed the submit button too early.
The issue here is that pt-query-digest should ignore certain types of C-style comments before attempting to analyse the query for table and schema names, as this might cause issues with whatever is in the comment.

The comment in this query was automatically generated by Java Hibernate framework, and contains the SQL operation being attempted and the caller method's full-qualified class name, java-style.

[Cees de Groot (casedeg)]

Something like the following could do the trick. However, I'm not sure how it would impact the tool in general (quick ugly diff on the binary, but location should be clear).

*** /usr/local/bin/pt-query-digest 2013-02-15 13:13:10.000000000 +0100
--- /tmp/patched 2013-02-15 13:12:37.000000000 +0100
***************
*** 4907,4912 ****
--- 4907,4915 ----
           else {
              PTDEBUG && _d("Got the query/arg line");
              my $arg = substr($stmt, $pos - length($line));
+ # Strip C-style comments
+ $arg =~ s#/\*[^*]*\*+([^/*][^*]*\*+)*/|("(\\.|[^"\\])*"|'(\\.|[^'\\])*'|.[^/"'\\]*)#defined $2 ? $2 : ""#gse;
+ PTDEBUG && _d('Clean query = ', $arg);
              push @properties, 'arg', $arg, 'bytes', length($arg);
              if ( $args{misc} && $args{misc}->{embed}
                 && ( my ($e) = $arg =~ m/($args{misc}->{embed})/)

[Daniel Nichter (daniel-nichter)]

pqd does attempt to handle /* comments */, but in this case it gets it wrong.

[Shahriyar Rzayev (rzayev-sehriyar)]

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PT-602