Based on #879076: Do not set expired=0 where expired is already 0..

A tweak to a SQL query to make the SQL server not try to update so many rows.

CommentFileSizeAuthor
#4 sql-performance-opt-D6.patch735 bytesGarrett Albright
sql-performance-opt.patch735 bytesGarrett Albright

Comments

neclimdul’s picture

neclimdul
He/Him
commented

Seems a reasonable optimization.

pwolanin’s picture

pwolanin commented
Status: Active » Needs review

This is pretty trivial as patches go, though I'm not sure it makes any difference for mysql.

Status: Needs review » Needs work

The last submitted patch, sql-performance-opt.patch, failed testing.

Garrett Albright’s picture

Garrett Albright commented
Status: Needs work » Needs review
StatusFileSize
new sql-performance-opt-D6.patch735 bytes

Rename to stop testbot from trying to test a D6 patch.

pwolanin’s picture

pwolanin commented
Status: Needs review » Needs work

Per the related core issue, is not clear that this change provides any performance benefit. Setting to CNW until we see a benchmark.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Closed (duplicate)

We don't need two existing issues. I've marked #879076: Do not set expired=0 where expired is already 0. as needing backport to Drupal Queue.