Current when ever we write to watchdog we wait for getLasterror to determine if we want to create a collection or not this blocks writes. Instead we can check if the id exists this does not then block the operation waiting on a write.

CommentFileSizeAuthor
#5 0001-Issue-2056815-by-marcingy-fgm-backport-performance-i.patch5.96 KBfgm
#1 watchdog.patch1.69 KBmarcingy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Active » Needs review
FileSize
watchdog.patch1.69 KB
jyee’s picture

jyee CreditAttribution: jyee commented
Status: Needs review » Fixed

Looks like chx committed this in c8d9d69. Marking as fixed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm at OSInet commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Issue summary: View changes
Status: Closed (fixed) » Patch (to be ported)

Needs backport.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm at OSInet commented
Status: Patch (to be ported) » Needs review
FileSize
0001-Issue-2056815-by-marcingy-fgm-backport-performance-i.patch5.96 KB

Backported, tests pass locally: db contents seem sane.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm at OSInet commented
Parent issue: » #2495763: Prepare 6.x (final) release.

  • fgm committed 21ebb8c on 6.x-1.x 
    Issue #2056815 by marcingy, fgm: backport performance improvements from...
fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm at OSInet commented
Status: Needs review » Fixed

Completed and committed to 6.x-1.x, thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.