Patch #33431
openBetter performance for Time entries without issue and activity filters
0%
Description
At the moment all time entry queries join issue and activities. If there is a large number of time entries and issues, this is a big performance issue.
Some of our customers have more than 500.000 time entries and even more issues. A query without a time filter requires minutes. With this attached patch this improves it to some seconds.
Issue and activity joins are only add, if they are required.
It would be great, to get this in Redmine.
Files
Updated by Go MAEDA over 4 years ago
- Category changed from Time tracking to Performance
- Target version set to Candidate for next major release
Updated by Alexander Meindl over 4 years ago
My patch has problems with column sort order.
But I found the real problem now. It is a missing condition to project_id for Enumeration. I am working on a new patch to fix this problem. Please do not commit the current patch.
Updated by Marius BĂLTEANU over 4 years ago
Alexander Meindl wrote:
My patch has problems with column sort order.
But I found the real problem now. It is a missing condition to project_id for Enumeration. I am working on a new patch to fix this problem. Please do not commit the current patch.
Nice feature, but we should have tests for this change.
Updated by Alexander Meindl over 4 years ago
Here is my new fixed version.
Adding
where("#{Enumeration.table_name}.project_id = #{TimeEntry.table_name}.project_id")
fixed the problem. But I moved activity join from base_scope to results_scope, because as I understood it, activity join is not required for totals and count.
I am not sure, how I can test this change with an additional test. Existing test should work, of course.
Updated by Alexander Meindl over 4 years ago
And again, now all tests are running.
With my test data, /time_entries without filters
- without patch: ActiveRecord: 345038.1ms
- with patch ActiveRecord: 6038.4ms
This is not perfect, but way better than before. At the moment I have no idea how it can be more improved. The problem is table layout of enumerations table.
Updated by Alexander Meindl about 2 years ago
2 years later, a new patch introduce skipping joins for count time entries, which are not required. Test added.
All tests run successful with this changes.
Updated by Alexander Meindl about 2 years ago
New version of patch has some optimization for count with activities.
Updated by Omega Code about 2 years ago
on the ther hand (in regard to #37962) this patch helped a lot under Redmine 4.2.8 (mysql 8):
time for time entries queries was reduced approx by 40% for me.
thank you!
Updated by John Ramsden almost 2 years ago
We have hundreds of thousands of time entries and redmine became almost unusable recently, this patch made a huge difference to us. It would be great if it can be pushed into an upcoming release.
Updated by Go MAEDA almost 2 years ago
- Target version changed from Candidate for next major release to 5.1.0
Updated by sly sly over 1 year ago
I have a problem with the v5 patch.
If you move an issue to another project, the project_id no longer matches the request and is ignored when displaying the numbers.
An example:
Issue 2982 belonged to project with ID = 8 when the time entry 34446 was created and then was moved in another project ID = 17
MariaDB [redmine_import]> select te.*,projects.id,enumerations.id,enumerations.project_id,enumerations.type,users.id,users.type,projects.id FROM time_entries AS te INNER JOIN `projects` ON `projects`.`id` = te.project_id INNER JOIN enumerations ON enumerations.id = te.activity_id AND enumerations.type = 'TimeEntryActivity' INNER JOIN users ON users.id = te.user_id AND users.type IN ('User', 'AnonymousUser') WHERE te.id=34446;
+-------+------------+-----------+---------+----------+-------+------------------------------------+-------------+------------+-------+--------+-------+---------------------+---------------------+---------+------+----+-----+------------+-------------------+----+------+----+
| id | project_id | author_id | user_id | issue_id | hours | comments | activity_id | spent_on | tyear | tmonth | tweek | created_on | updated_on | rate_id | cost | id | id | project_id | type | id | type | id |
+-------+------------+-----------+---------+----------+-------+------------------------------------+-------------+------------+-------+--------+-------+---------------------+---------------------+---------+------+----+-----+------------+-------------------+----+------+----+
| 34446 | 17 | 12 | 12 | 2982 | 4 | Fix alma container issue for devel | 172 | 2021-10-04 | 2021 | 10 | 40 | 2021-10-05 07:59:26 | 2021-10-05 07:59:26 | NULL | NULL | 17 | 172 | 8 | TimeEntryActivity | 12 | User | 17 |
+-------+------------+-----------+---------+----------+-------+------------------------------------+-------------+------------+-------+--------+-------+---------------------+---------------------+---------+------+----+-----+------------+-------------------+----+------+----+
1 row in set (0.001 sec)
So the DB queries was KO and doesn't display the good numbers, for an hotfix we just removed
def base_result_scope
base_scope.joins(:activity, :user)
.includes(:activity)
.left_join_issue
# XXX - remove to let time entry report working fine
# .where(activity_scope_sql)
end
Updated by Go MAEDA about 1 year ago
- Target version changed from 5.1.0 to 6.0.0
Updated by Kaustubh Patil 7 months ago
Hello There,
We are facing the issue while loading global spent time. Can the above mentioned patches help us to mitgate our issue. Following are the details of our Redmine Environment:
Environment:
Redmine version 5.1.2.stable
Ruby version 3.2.4-p170 (2024-04-23) [x86_64-linux]
Rails version 6.1.7.7
Environment production
Database adapter Mysql2
Mailer queue ActiveJob::QueueAdapters::SidekiqAdapter
Mailer delivery smtp
Redmine settings:
Redmine theme Zenmine-510 (includes JavaScript)
SCM:
Subversion 1.14.2
Mercurial 6.3.2
Bazaar 3.3.2
Git 2.39.2
Filesystem
Redmine plugins:
redmine_agile 1.6.9
redmine_budgets 1.0.5
redmine_checklists 3.1.23
redmine_contacts 4.4.1
redmine_contacts_helpdesk 4.2.2
redmine_contacts_invoices 4.2.9
redmine_reporter 2.0.4
We have tried installing the time_query_performace_v5.patch but things don't seem to work. Pls guide us on the same to resolve this issue.
Updated by Kaustubh Patil 7 months ago
Hello there, could anyone pls provide any insights on this.
Updated by Marius BĂLTEANU about 2 months ago
- Target version changed from 6.0.0 to 6.1.0