[drupal-devel] [feature] Sort logs by ID instead of timestamp

killes drupal-devel at drupal.org
Thu Aug 4 13:36:07 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    watchdog.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  njivy
 Updated by:   killes at www.drop.org
-Status:       patch (code needs review)
+Status:       patch (ready to be committed)
 Attachment:   http://drupal.org/files/issues/wid.patch (782 bytes)

According to
auto_increments ususally do not get re-used.

The patch is attached and tested.

killes at www.drop.org

Previous comments:

Wed, 25 May 2005 01:59:52 +0000 : njivy

In watchdog.module I propose changing the key used to sort logs
chronologically.  Instead of using the timestamp, which is accurate
only to the nearest second, we could use the log entry id ("wid" in the
database).  As I understand it, the IDs are assigned sequentially, so
this is a good approximation to the correct order in which the events
are recorded.  Also, I don't know of any events that are recorded after
a delay.

The problem with the current sorting key arises when logged events come
within 1 second of each other.  The order of events is not always
preserved when viewing the log because the timestamp is identical for
several events.

In "modules/watchdog.module" neard line 91, the word "timestamp" is
replaced with "wid":
$header = array(
    ' ',
    array('data' => t('Type'), 'field' => 'w.type'),
    array('data' => t('Date'), 'field' => 'w.wid', 'sort' =>
    array('data' => t('Message'), 'field' => 'w.message'),
    array('data' => t('User'), 'field' => 'u.name'),
    array('data' => t('Operations'), 'colspan' => '2')

Preliminary testing suggests this is a good tweak.


Wed, 25 May 2005 03:53:05 +0000 : Steven

Do auto_increment id's never go back to fill in deleted rows? We need to
make sure on pgsql and mysql.

Another option might be to sort by timestamp first, then by wid. I'm
not sure if the current tablesort supports that though.


Thu, 26 May 2005 18:32:39 +0000 : Dries

Also, I think auto_increment IDs rotate/overflow.  This isn't very
likely though.  Currently, drupal.org's largest wid value is appr.

More information about the drupal-devel mailing list