[#9161] - [com_content] Boost sql query by skipping join to #__content_frontage if not needed
- Fixed in Code Base
- 8 May 2016
- Medium
- Build: staging
- # 9161
- Diff
- csthomas:skipfrontpage
- Pending continuous-integration/travis-ci/pr The Travis CI build is in progress Details
User tests: Successful: Unsuccessful:
Sql query generated by components/com_content/models/articles.php is usually heavy,
I propose to skip sql join to #__content_frontpage table if it is not used on category view or archive view.
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
| Category | ⇒ | SQL |
As an author, I can say that this patch is one of several others that I use on production site (more than 100k articles) for long time without any problems, but I'm interested in any comments.
How to test on front page:
- go to com_content category view
- go to com_content categories view
- go to featured view
- go to archive view
Test results of modules based on cm_content models:
- mod_articles_...
| Title |
|
||||||
Explanation:
When articles do not require order by front page ordering then we can skip join and use featured column only in where clause.
I have tested this item 
successfully on 410d108
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
This PR has received new commits.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
I have tested this item successfully on 2de59c4
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
| Status | Pending | ⇒ | Ready to Commit |
RTC. Thanks for coding and testing.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Pending |
| Labels | |||
Reset to pending for one extra check
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
| Labels |
Removed:
?
|
||
I have tested this item successfully on 2de59c4
I can't see problems on the blog view. So i would give this a go.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
| Status | Pending | ⇒ | Ready to Commit |
Thank you @csthomas!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-05-08 13:08:29 |
| Closed_By | ⇒ | Kubik-Rubik |
| Labels |
Removed:
?
|
||
Can anyone please elaborate on how to test it.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9161.