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

CommentFileSizeAuthor
aggregator_count.patch591 bytesandypost

Comments

andypost’s picture

andypost
he/him
Primary language Russian
commented

Any reviews? Patch so simple...

aggregator_count.patch queued for re-testing.

twistor’s picture

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
commented
Title: Optimize count(has) quiery » Optimize count(has) query
lars toomre’s picture

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 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 commented
Issue tags: +Performance, +Administration