diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2022-04-08 19:35:31 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-08 19:35:31 +0200 |
commit | 68273a7c6d6c630b6c88764579580682e12eebce (patch) | |
tree | 5d4c17b01c5088506c074f09fab6ed51b38a3703 /app/models/trends/links.rb | |
parent | a39bf04fe6b6a98547737cc15f42727428376e67 (diff) |
Fix dangling language-specific trends (#17997)
- Change score half-life for trending statuses from 2 to 6 hours - Change score threshold for trimming old items from 1 to 0.3
Diffstat (limited to 'app/models/trends/links.rb')
-rw-r--r-- | app/models/trends/links.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/models/trends/links.rb b/app/models/trends/links.rb index 62308e706..5f046643a 100644 --- a/app/models/trends/links.rb +++ b/app/models/trends/links.rb @@ -30,7 +30,6 @@ class Trends::Links < Trends::Base def refresh(at_time = Time.now.utc) preview_cards = PreviewCard.where(id: (recently_used_ids(at_time) + currently_trending_ids(false, -1)).uniq) calculate_scores(preview_cards, at_time) - trim_older_items end def request_review @@ -101,6 +100,8 @@ class Trends::Links < Trends::Base }) end + trim_older_items + # Clean up localized sets by calculating the intersection with the main # set. We do this instead of just deleting the localized sets to avoid # having moments where the API returns empty results @@ -108,7 +109,7 @@ class Trends::Links < Trends::Base redis.pipelined do Trends.available_locales.each do |locale| redis.zinterstore("#{key_prefix}:all:#{locale}", ["#{key_prefix}:all:#{locale}", "#{key_prefix}:all"], aggregate: 'max') - redis.zinterstore("#{key_prefix}:allowed:#{locale}", ["#{key_prefix}:allowed:#{locale}", "#{key_prefix}:all"], aggregate: 'max') + redis.zinterstore("#{key_prefix}:allowed:#{locale}", ["#{key_prefix}:allowed:#{locale}", "#{key_prefix}:allowed"], aggregate: 'max') end end end |