From 079e1108f118c5a4acb3ff2a96df026011909e97 Mon Sep 17 00:00:00 2001 From: Martin Brennan Date: Fri, 27 Jan 2023 16:32:11 +1000 Subject: [PATCH] DEV: Remove stale TODO (#20039) This TODO is irrelevant -- in reality this has not been a perf issue, and there is not actually an N1 here. Furthermore, this is only used in a single plugin, not in core. --- app/models/topic.rb | 2 -- 1 file changed, 2 deletions(-) diff --git a/app/models/topic.rb b/app/models/topic.rb index e373f4f4d4a..b1e764b7a97 100644 --- a/app/models/topic.rb +++ b/app/models/topic.rb @@ -1895,8 +1895,6 @@ class Topic < ActiveRecord::Base def incoming_email_addresses(group: nil, received_before: Time.zone.now) email_addresses = Set.new - # TODO(martin) Look at improving this N1, it will just get slower the - # more replies/incoming emails there are for the topic. self .incoming_email .where("created_at <= ?", received_before)