Bug #59861 | why does log_slow_statement call thd_proc_info twice? | ||
---|---|---|---|
Submitted: | 1 Feb 2011 1:50 | Modified: | 7 Mar 2011 23:18 |
Reporter: | Mark Callaghan | Email Updates: | |
Status: | Closed | Impact on me: | |
Category: | MySQL Server: Logging | Severity: | S3 (Non-critical) |
Version: | 5.1, 5.5, trunk | OS: | Any |
Assigned to: | Jon Olav Hauglid | CPU Architecture: | Any |
Tags: | log, query, slow |
[1 Feb 2011 1:50]
Mark Callaghan
[1 Feb 2011 5:02]
Valeriy Kravchuk
Verified by code review.
[3 Feb 2011 9:03]
Bugs System
A patch for this bug has been committed. After review, it may be pushed to the relevant source trees for release in the next version. You can access the patch from: http://lists.mysql.com/commits/130303 3300 Jon Olav Hauglid 2011-02-03 Bug #59861 why does log_slow_statement call thd_proc_info twice? This patch removes an extraneous thd_proc_info(thd, "logging slow query") call from low_slow_statement(). The removed call was from before it was decided to actually log the query to the slow log. No test case added.
[3 Feb 2011 9:13]
Magne Mæhre
Ok to push
[7 Feb 2011 11:49]
Bugs System
A patch for this bug has been committed. After review, it may be pushed to the relevant source trees for release in the next version. You can access the patch from: http://lists.mysql.com/commits/130557 3603 Jon Olav Hauglid 2011-02-07 Bug #59861 why does log_slow_statement call thd_proc_info twice? This patch removes an extraneous thd_proc_info(thd, "logging slow query") call from low_slow_statement(). The removed call was from before it was decided to actually log the query to the slow log. No test case added.
[7 Feb 2011 11:50]
Bugs System
Pushed into mysql-trunk 5.6.2 (revid:jon.hauglid@oracle.com-20110207114918-m0a336hi7wav0wmt) (version source revid:jon.hauglid@oracle.com-20110207114918-m0a336hi7wav0wmt) (merge vers: 5.6.2) (pib:24)
[7 Mar 2011 23:18]
Paul DuBois
Code cleanup. No user impact. No changelog entry needed.
[7 Mar 2011 23:20]
Paul DuBois
CHANGESET - http://lists.mysql.com/commits/130557
[29 Oct 2011 10:14]
MySQL Verification Team
Seems thd->current_utime() is called prematurely also. I've noted that in a new internal bug report.