From ff0edc35406f2b567155c5d303b87febb3edf2e3 Mon Sep 17 00:00:00 2001
From: rhymes <rhymes@hey.com>
Date: Fri, 4 Sep 2020 20:40:53 +0200
Subject: [PATCH] [deploy] Remove draft articles with a duplicate canonical_url
 (#10138)

---
 ...t_articles_with_duplicate_canonical_url.rb | 78 +++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100644 lib/data_update_scripts/20200901085230_remove_draft_articles_with_duplicate_canonical_url.rb

diff --git a/lib/data_update_scripts/20200901085230_remove_draft_articles_with_duplicate_canonical_url.rb b/lib/data_update_scripts/20200901085230_remove_draft_articles_with_duplicate_canonical_url.rb
new file mode 100644
index 000000000..7e1d6e3cd
--- /dev/null
+++ b/lib/data_update_scripts/20200901085230_remove_draft_articles_with_duplicate_canonical_url.rb
@@ -0,0 +1,78 @@
+module DataUpdateScripts
+  class RemoveDraftArticlesWithDuplicateCanonicalUrl
+    def run
+      # Currently there are duplicate draft articles in the DB with the same canonical_url.
+
+      # This statement deletes all draft articles in excess found to be duplicate over canonical_url,
+      # excluding those whose body_markdown is different from the other duplicate occurrences
+      result = ActiveRecord::Base.connection.execute(
+        <<~SQL,
+          WITH duplicates_draft_articles AS
+              (SELECT id
+               FROM
+                   (SELECT id,
+                           published,
+                           body_markdown,
+                           LAG(body_markdown, 1) OVER(PARTITION BY canonical_url
+                                                      ORDER BY id ASC) AS previous_body_markdown,
+                           ROW_NUMBER() OVER(PARTITION BY canonical_url
+                                             ORDER BY id ASC) AS row_number
+                    FROM articles
+                    WHERE canonical_url IS NOT NULL ) duplicates
+               WHERE duplicates.row_number > 1
+                   AND published = 'f' -- drafts
+                   AND body_markdown = previous_body_markdown -- with the same body
+           )
+          DELETE
+          FROM articles
+          WHERE id IN (SELECT id FROM duplicates_draft_articles) RETURNING id;
+        SQL
+      )
+
+      # Sending IDs of deleted articles to Datadog
+      result.map { |row| row["id"] }.in_groups_of(1000) do |ids|
+        DatadogStatsClient.event(
+          "DataUpdateScripts::RemoveDraftArticlesWithDuplicateCanonicalUrl",
+          "deleted draft articles with the same canonical_url and same body_markdown",
+          tags: ids,
+        )
+      end
+
+      # Now that all duplicates with the same body are gone, we need to deal with duplicate canonical URLs
+      # with different bodies.
+      # We thus select the oldest for removal preserving the most recent one
+      result = ActiveRecord::Base.connection.execute(
+        <<~SQL,
+          WITH duplicates_draft_articles AS
+              (SELECT id
+               FROM
+                   (SELECT id,
+                           published,
+                           body_markdown,
+                           LAG(body_markdown, 1) OVER(PARTITION BY canonical_url
+                                                      ORDER BY created_at DESC) AS previous_body_markdown,
+                           ROW_NUMBER() OVER(PARTITION BY canonical_url
+                                             ORDER BY created_at DESC) AS row_number
+                    FROM articles
+                    WHERE canonical_url IS NOT NULL ) duplicates
+               WHERE duplicates.row_number > 1
+                   AND published = 'f' -- drafts
+                   AND body_markdown != previous_body_markdown -- with different bodies
+           )
+          DELETE
+          FROM articles
+          WHERE id IN (SELECT id FROM duplicates_draft_articles) RETURNING id;
+        SQL
+      )
+
+      # Sending IDs of deleted articles to Datadog
+      result.map { |row| row["id"] }.in_groups_of(1000) do |ids|
+        DatadogStatsClient.event(
+          "DataUpdateScripts::RemoveDraftArticlesWithDuplicateCanonicalUrl",
+          "deleted draft articles with the same canonical_url and different body_markdown",
+          tags: ids,
+        )
+      end
+    end
+  end
+end