Ticket #5179 (closed defect (bug): fixed)

Opened 4 years ago

Last modified 4 years ago

Use INNER JOIN instead of LEFT JOIN for taxonomy

Reported by: ryan Owned by: anonymous
Priority: normal Milestone: 2.3.1
Component: General Version: 2.3
Severity: normal Keywords: taxonomy performance
Cc:

Description

We really don't need LEFT JOIN as far as I can tell. Let's use the faster INNER JOIN.

Attachments

left_join.diff Download (5.6 KB) - added by ryan 4 years ago.

Change History

ryan — 4 years ago

comment:1   ryan — 4 years ago

  • Status changed from new to closed
  • Resolution set to fixed

(In [6224]) Use inner joins instead of left joins for taxonomy. Inner is faster. fixes #5179

comment:2   ryan — 4 years ago

  • Milestone changed from 2.4 to 2.3.1

Nominating for 2.3.1. We might like the performance bump there.

comment:3   ryan — 4 years ago

(In [6253]) Use inner joins instead of left joins for taxonomy. Inner is faster. fixes #5179 for 2.3

comment:4   ryan — 4 years ago

Decided to go for it for 2.3. We've been using this on wordpress.com without incident and it does create a nice performance bump.

Note: See TracTickets for help on using tickets.