Overview: Link

Patch attached that forces watchdog inserts to generate sequence IDs using the sequence table rather than the auto_increment feature..

Thanks
Karthik.

CommentFileSizeAuthor
bootstrap_16.patch865 bytesZen

Comments

killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Needs review » Needs work

See my reply in the other issue. An auto_increment is usually faster. I am empted to mark this "won't fix". In any case, the changes to the database definition file and the neccessary upgrade should be included.

Zen’s picture

Zen commented
Status: Needs work » Needs review

This should work fine without the need to update the db definition file. i.e. it will work fine even if the auto_increment remains (if pointless)..

-K

jose reyero’s picture

jose reyero commented

-1 to add more queries for watchdog.

I agree with Killes, and using the sequences table would mean a pair more queries.
This is somehow related to this other one, http://drupal.org/node/32576, which is about performance impact of watchdog.

Zen’s picture

Zen commented
Status: Needs review » Closed (fixed)

http://drupal.org/node/38585