php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #62187 The MySQL driver does not store a timestamp for an event [+PATCH]
Submitted: 2012-05-30 08:13 UTC Modified: 2012-06-08 08:28 UTC
From: mathieu at mollie dot nl Assigned: patrickallaert (profile)
Status: Closed Package: APM (PECL)
PHP Version: Irrelevant OS: Irrelevant
Private report: No CVE-ID: None
View Add Comment Developer Edit
 [2012-05-30 08:13 UTC] mathieu at mollie dot nl 
Description:
------------
While the SQLITE3 driver stores a value in the TS column, the MYSQL driver does 
not store anything. Additionally, the MySQL create.sql creates the TS column as a 
TEXT instead of a DATETIME.

Patch available at https://github.com/MatMol/php-
apm/commit/b6721825a17757fb27869f9a06aa1dd689048514

This patch also fixes issue 62142, by using the column. 


Test script:
---------------
* Install & activate APM
* Cause error
* Inspect database



Expected result:
----------------
* See some kind of date 

Actual result:
--------------
* See column 'ts' being empty

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2012-06-04 16:03 UTC] patrickallaert@php.net 
Automatic comment from SVN on behalf of patrickallaert
Revision: http://svn.php.net/viewvc/?view=revision&revision=325973
Log: Fixed Bug #62187: The MySQL driver does not store a timestamp for an event

Thanks to mathieu at mollie dot nl.
 [2012-06-07 21:29 UTC] patrickallaert@php.net
-Assigned To: +Assigned To: patrickallaert
 [2012-06-07 21:44 UTC] patrickallaert@php.net
-Status: Assigned +Status: Closed
 [2012-06-07 21:44 UTC] patrickallaert@php.net 
Thank you for your bug report. This issue has already been fixed
in the latest released version of PHP, which you can download at 
http://www.php.net/downloads.php

Fixed in APM 1.1.0beta4
 [2012-06-08 07:23 UTC] mathieu at mollie dot nl 
Same issue exists in slow_request,
 [2012-06-08 07:23 UTC] mathieu at mollie dot nl
-Status: Closed +Status: Assigned
 [2012-06-08 07:55 UTC] patrickallaert@php.net 
Automatic comment from SVN on behalf of patrickallaert
Revision: http://svn.php.net/viewvc/?view=revision&revision=326031
Log: Fixed Bug #62187: The MySQL driver does not store a timestamp for an event

Thanks to mathieu at mollie dot nl.
 [2012-06-08 08:28 UTC] patrickallaert@php.net
-Status: Assigned +Status: Closed
 [2012-06-08 08:28 UTC] patrickallaert@php.net 
Thank you for your bug report. This issue has already been fixed
in the latest released version of PHP, which you can download at 
http://www.php.net/downloads.php

That was fixed in the release, but the code was not yet pushed to SVN.

Both "ts" fields are TIMESTAMPs and doesn't require to be explicitly set in 
INSERT.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Tue Apr 23 23:01:29 2024 UTC