Loading...

XML

Word

Printable

Type: Improvement
Resolution: Won't Fix
Priority: Minor
Fix Version/s: None
Affects Version/s: 3.0.2
Component/s: Administration, Database SQL/XMLDB
Labels:
- triaged

Database:

PostgreSQL
Affected Branches:
MOODLE_30_STABLE
Pull from Repository:
https://github.com/grigory-catalyst/moodle
Pull Main Branch:
~~MDL-52801~~_Feature_group_concat
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...grigory-catalyst:MDL-52801_Feature_group_concat
Workaround:

Hide

There is a unit test which compares the output of both classes old and new one.

Show
There is a unit test which compares the output of both classes old and new one.
Testing Instructions:

Hide

run
vendor/bin/phpunit admin/tool/health/tests.php
vendor/bin/phpunit admin/tool/health/tests.php

Show
run vendor/bin/phpunit admin/tool/health/tests.php vendor/bin/phpunit admin/tool/health/tests.php

problem_000013 class from admin/tool/health/index.php sends an unoptimal database query which makes N*M scans in the database (N,M - number of rows in

{question}

{question_multianswer}

accordingly) which is very time-consuming process.

TO REPRODUCE:
create 500k records in both tables and try to run
/admin/tool/health/index.php script

WHAT EXPECTED:
linear complexity at least. Execution time < 30 seconds

WHAT HAPPENED:
N^2 complexity. Execution time > 900 seconds

has a non-specific relationship to

MDL-52817 New sql_group_concat db method

Closed

has been marked as being related by

MDL-67818 Create a Check API

Closed

will be (partly) resolved by

MDL-67228 Set tool_replace maturity

Closed

Assignee:: Unassigned

Reporter:: Grigory Baleevskiy

Participants:: CiBoT, Grigory Baleevskiy, Marina Glancy, Matt Porritt, Paul Holden, Tim Hunt

Votes:: 1 Vote for this issue

Watchers:: 8 Start watching this issue

Created:: 18/Jan/16 6:48 PM

Updated:: 21/Jul/23 5:20 PM

Resolved:: 21/Jul/23 5:20 PM

Details

Description

Attachments

Issue Links

Activity

People

Dates

Clockify