Bug #22285 | Community: SHOW PROFILE (thd->proc_info) | ||
---|---|---|---|
Submitted: | 12 Sep 2006 21:49 | Modified: | 9 Nov 2006 21:22 |
Reporter: | Marc ALFF | Email Updates: | |
Status: | Closed | Impact on me: | |
Category: | MySQL Server | Severity: | S4 (Feature request) |
Version: | 5.2 | OS: | |
Assigned to: | Marc ALFF | CPU Architecture: | Any |
[12 Sep 2006 21:49]
Marc ALFF
[12 Sep 2006 22:04]
Marc ALFF
The issue found is a code cleanup issue. Reproduced and Verified by code review (Marc Alff, Konstantin Osipov).
[4 Oct 2006 20:31]
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/13092 ChangeSet@1.2333, 2006-10-04 14:31:24-07:00, malff@weblab.(none) +20 -0 Bug#22285 (Community: SHOW PROFILE (thd->proc_info) This is a minor code cleanup change, no functionality is affected. Modifications to thd->proc_info have been revised to use THD_SET_PROC_INFO instead, which provide an extra trace in debug mode. Note that thd->proc_info is a public member, used by some storage engines. The storage engine code has not been changed, as this access to THD data needs to be revised for a different reason (engines plug-in). In non debug builds, THD_SET_PROC_INFO expands to the original code.
[1 Nov 2006 9:10]
Konstantin Osipov
The patch is approved. While reviewing, I assumed that it does what is required for SHOW PROFILE and only was making sure nothing is broken by the patch.
[2 Nov 2006 17:18]
Marc ALFF
Pushed to mysql-5.2
[9 Nov 2006 21:22]
Konstantin Osipov
No documentation entry is needed, the patch is in 5.2 main tree.
[2 Jan 2007 19:27]
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/17539 ChangeSet@1.2326, 2007-01-02 14:26:42-05:00, cmiller@zippy.cornsilk.net +17 -0 Bug#22285: Community: SHOW PROFILE (thd->proc_info) BACKPORTED from 5.2 Enterprise. This is a minor code cleanup change, no functionality is affected. Modifications to thd->proc_info have been revised to use THD_SET_PROC_INFO instead, which provide an extra trace in debug mode. Note that thd->proc_info is a public member, used by some storage engines. The storage engine code has not been changed, as this access to THD data needs to be revised for a different reason (engines plug-in). In non debug builds, THD_SET_PROC_INFO expands to the original code.