Actions
Patch #36503
closed
Reduce extra queries in IssueQuery.default
Start date:
Due date:
% Done:
0%
Estimated time:
Description
The attached patch reduces the total number of SQL queries during the execution of IssueQuery.default.
No default queries in any levels:
before:
UserPreference Load (0.2ms) SELECT "user_preferences".* FROM "user_preferences" WHERE "user_preferences"."user_id" = ? LIMIT ? [["user_id", 1], ["LIMIT", 1]] ↳ app/models/user.rb:394:in `pref' IssueQuery Load (0.1ms) SELECT "queries".* FROM "queries" WHERE "queries"."type" = ? AND "queries"."id" IS NULL LIMIT ? [["type", "IssueQuery"], ["LIMIT", 1]] ↳ app/models/issue_query.rb:83:in `default' Setting Load (0.1ms) SELECT "settings".* FROM "settings" WHERE "settings"."name" = ? ORDER BY "settings"."id" DESC LIMIT ? [["name", "default_issue_query"], ["LIMIT", 1]] ↳ app/models/setting.rb:360:in `find_or_default' IssueQuery Load (0.1ms) SELECT "queries".* FROM "queries" WHERE "queries"."type" = ? AND "queries"."id" = ? LIMIT ? [["type", "IssueQuery"], ["id", nil], ["LIMIT", 1]] ↳ app/models/issue_query.rb:86:in `default'
after:
UserPreference Load (0.1ms) SELECT "user_preferences".* FROM "user_preferences" WHERE "user_preferences"."user_id" = ? LIMIT ? [["user_id", 1], ["LIMIT", 1]] ↳ app/models/user.rb:394:in `pref' Setting Load (0.1ms) SELECT "settings".* FROM "settings" WHERE "settings"."name" = ? ORDER BY "settings"."id" DESC LIMIT ? [["name", "default_issue_query"], ["LIMIT", 1]] ↳ app/models/setting.rb:360:in `find_or_default'
A project-level default query defined:
before:
UserPreference Load (0.2ms) SELECT "user_preferences".* FROM "user_preferences" WHERE "user_preferences"."user_id" = ? LIMIT ? [["user_id", 1], ["LIMIT", 1]] ↳ app/models/user.rb:394:in `pref' IssueQuery Load (0.2ms) SELECT "queries".* FROM "queries" WHERE "queries"."type" = ? AND "queries"."id" IS NULL LIMIT ? [["type", "IssueQuery"], ["LIMIT", 1]] ↳ app/models/issue_query.rb:83:in `default' IssueQuery Load (0.2ms) SELECT "queries".* FROM "queries" WHERE "queries"."type" = ? AND "queries"."id" = ? LIMIT ? [["type", "IssueQuery"], ["id", 4], ["LIMIT", 1]] ↳ app/models/issue_query.rb:85:in `default'
after:
UserPreference Load (0.2ms) SELECT "user_preferences".* FROM "user_preferences" WHERE "user_preferences"."user_id" = ? LIMIT ? [["user_id", 1], ["LIMIT", 1]] ↳ app/models/user.rb:394:in `pref' IssueQuery Load (0.2ms) SELECT "queries".* FROM "queries" WHERE "queries"."type" = ? AND "queries"."id" = ? LIMIT ? [["type", "IssueQuery"], ["id", 4], ["LIMIT", 1]] ↳ app/models/issue_query.rb:87:in `default'
Files
Related issues
Updated by Go MAEDA almost 3 years ago
- Related to Feature #7360: Issue custom query: default query per instance, project and user added
Updated by Go MAEDA almost 3 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed the patch as a part of #7360.
Updated by Dmitry Makurin almost 3 years ago
Isn't there https://www.redmine.org/projects/redmine/repository/entry/trunk/app/models/issue_query.rb#L83 also supposed to be a present? at the end of second condition?
If (query_id = user.pref.default_issue_query) equals to "" the query stil fires.
diff --git a/app/models/issue_query.rb b/app/models/issue_query.rb
index 5a3e14474..e7ac1c821 100644
--- a/app/models/issue_query.rb
+++ b/app/models/issue_query.rb
@@ -80,7 +80,7 @@ class IssueQuery < Query
def self.default(project: nil, user: User.current)
query = nil
# user default
- if user&.logged? && (query_id = user.pref.default_issue_query)
+ if user&.logged? && (query_id = user.pref.default_issue_query).present?
query = find_by(id: query_id)
end
# project default
Updated by Marius BĂLTEANU over 2 years ago
- Status changed from Closed to Reopened
Updated by Marius BĂLTEANU over 2 years ago
- Status changed from Reopened to Closed
Dmitry Makurin wrote:
Isn't there https://www.redmine.org/projects/redmine/repository/entry/trunk/app/models/issue_query.rb#L83 also supposed to be a
present?at the end of second condition?If
(query_id = user.pref.default_issue_query)equals to "" the query stil fires.[...]
Committed the patch, thanks for pointing this out.
Updated by Dmitry Makurin over 2 years ago
- Copied to Patch #37135: Reduce extra queries in ProjectQuery.default added
Actions