? Pending

User tests: Successful: Unsuccessful:

avatar csthomas
csthomas
9 May 2018

Pull Request for Issue #13265.

Summary of Changes

  1. This PR makes the query cache disabled when db driver runs in debug mode.
  2. Enable Profile for Mysql PDO.

Not direct forked but based on #13268.

Testing Instructions

Details at #13268

  1. Enable debug mode in global config and system debug plugin
  2. Test at least one of driver: Mysql, MySqli or PDO Mysql.
  3. Check debug console "Database Queries" and "Profile" tab. Each query should run without using database cache.

Expected result

No cached query.
On PDOMYSQL driver profile tab is enabled.

Actual result

On debug mode queries use database cache.
On PDOMYSQL driver profile tab is disabled.

Documentation Changes Required

No

avatar csthomas csthomas - open - 9 May 2018
avatar csthomas csthomas - change - 9 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 May 2018
Category Libraries
avatar csthomas csthomas - change - 15 May 2018
Title
Disable sql query cache in debug mode
Enable Profiling for mysql (PDO) queries and disable cache in debug mode
avatar csthomas csthomas - edited - 15 May 2018
avatar csthomas csthomas - change - 15 May 2018
Title
Enable Profiling for mysql (PDO) queries and disable cache in debug mode
Enable Profiling tab for mysql (PDO) queries and disable cache in debug mode
avatar csthomas csthomas - edited - 15 May 2018
avatar csthomas csthomas - change - 30 Jun 2018
The description was changed
avatar csthomas csthomas - edited - 30 Jun 2018
avatar csthomas csthomas - change - 18 Jul 2018
The description was changed
avatar csthomas csthomas - edited - 18 Jul 2018
avatar csthomas csthomas - change - 19 Jul 2018
The description was changed
avatar csthomas csthomas - edited - 19 Jul 2018
avatar csthomas csthomas - change - 8 Aug 2018
The description was changed
avatar csthomas csthomas - edited - 8 Aug 2018
avatar alikon alikon - test_item - 8 Aug 2018 - Tested successfully
avatar alikon
alikon - comment - 8 Aug 2018

I have tested this item successfully on 8b23be5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20322.

avatar csthomas
csthomas - comment - 9 Sep 2018

At now it may be hard to follow the test instruction because the cache is disabled by default in mysql 5.7.

You will only see the difference in PDOMYSQL driver, in which the Profile tab will be visible.

To enable the cache, you can add the following lines to the mysql configuration file.
In case of Ubuntu 18 is /etc/mysql/mysql.conf.d/mysqld.cnf

query_cache_type = 1
query_cache_size = 10M
query_cache_limit = 10M 
avatar csthomas
csthomas - comment - 24 Sep 2018

This PR is designed to display more relevant information when the debug mode is enabled.

Maybe there is one more person to test any driver and do simple code review on the rest.

avatar SharkyKZ
SharkyKZ - comment - 15 Nov 2018

I have tested this item successfully on 8b23be5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20322.

avatar SharkyKZ
SharkyKZ - comment - 15 Nov 2018

I have tested this item successfully on 8b23be5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20322.

avatar SharkyKZ SharkyKZ - test_item - 15 Nov 2018 - Tested successfully
avatar Quy Quy - change - 15 Nov 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 15 Nov 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20322.

avatar mbabker mbabker - change - 17 Nov 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-11-17 17:33:14
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 17 Nov 2018
avatar mbabker mbabker - merge - 17 Nov 2018

Add a Comment

Login with GitHub to post a comment