Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#5179 closed defect (bug) (fixed)

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 (1)

left_join.diff​ (5.6 KB) - added by ryan 6 years ago.

Download all attachments as: .zip

Change History (5)

ryan — 6 years ago

comment:1   ryan — 6 years ago

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

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

comment:2   ryan — 6 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 — 6 years ago

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

comment:4   ryan — 6 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.