This place only ensures that category exists so should be converted to optimized "exist" query type

CommentFileSizeAuthor
aggregator_count.patch591 bytesandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented

Any reviews? Patch so simple...

aggregator_count.patch queued for re-testing.

twistor’s picture

twistor CreditAttribution: twistor commented
Status: Needs review » Reviewed & tested by the community

It works. I actually think it makes the intention of the code clearer as well. It checks that a category exists rather than how many exist.

oriol_e9g’s picture

oriol_e9g
Primary language Catalan
Location Barcelona
CreditAttribution: oriol_e9g commented
Title: Optimize count(has) quiery » Optimize count(has) query
Lars Toomre’s picture

Lars Toomre CreditAttribution: Lars Toomre commented
Title: Optimize count(has) query » Optimize count(has) quiery

This makes sense and helps avoid COUNT(*) query in MySQL Innodb tables (good!!).

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks.

Status: Fixed » Closed (fixed)

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

sun.core’s picture

sun.core CreditAttribution: sun.core commented
Issue tags: +Performance, +Administration