Disable RecordingsSameNameDifferentArtistsSameName

This causes daily.sh to hang on RunReports.pl, and there's no obvious way to optimize it yet.
metabrainz · May 21, 2020 · 5c30a1b · 5c30a1b
1 parent 5ae0316
commit 5c30a1b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/lib/MusicBrainz/Server/Report/RecordingsSameNameDifferentArtistsSameName.pm b/lib/MusicBrainz/Server/Report/RecordingsSameNameDifferentArtistsSameName.pm
@@ -4,6 +4,9 @@ use Moose;
 with 'MusicBrainz::Server::Report::RecordingReport',
      'MusicBrainz::Server::Report::FilterForEditor::RecordingID';
 
+# FIXME: This report has been disabled since the upgrade to PG 12, because
+# its query can no longer execute in under 5 minutes in production.
+
 sub query {
     "
     SELECT

diff --git a/lib/MusicBrainz/Server/ReportFactory.pm b/lib/MusicBrainz/Server/ReportFactory.pm
@@ -65,7 +65,6 @@ use MusicBrainz::Server::PagedReport;
     RecordingsWithoutVALink
     RecordingsWithEarliestReleaseRelationships
     RecordingsWithVaryingTrackLengths
-    RecordingsSameNameDifferentArtistsSameName
     RecordingTrackDifferentName
     ReleasedTooEarly
     ReleaseGroupsWithoutVACredit
@@ -146,7 +145,7 @@ use MusicBrainz::Server::Report::RecordingsWithoutVACredit;
 use MusicBrainz::Server::Report::RecordingsWithoutVALink;
 use MusicBrainz::Server::Report::RecordingsWithEarliestReleaseRelationships;
 use MusicBrainz::Server::Report::RecordingsWithVaryingTrackLengths;
-use MusicBrainz::Server::Report::RecordingsSameNameDifferentArtistsSameName;
+#use MusicBrainz::Server::Report::RecordingsSameNameDifferentArtistsSameName;
 use MusicBrainz::Server::Report::RecordingTrackDifferentName;
 use MusicBrainz::Server::Report::ReleasedTooEarly;
 use MusicBrainz::Server::Report::ReleaseGroupsWithoutVACredit;

diff --git a/root/report/ReportsIndex.js b/root/report/ReportsIndex.js
@@ -449,12 +449,12 @@ const ReportsIndex = ({$c}: {$c: CatalystContextT}) => (
                 to VA`)}
           </a>
         </li>
-        <li>
+        {/*<li>
           <a href="/report/RecordingsSameNameDifferentArtistsSameName">
             {l(`Recordings with the same name by different artists
                 with the same name`)}
           </a>
-        </li>
+        </li>*/}
         <li>
           <a href="/report/RecordingTrackDifferentName">
             {l('Recordings with a different name than their only track')}