Merge pull request Ericsson#4358 from dkrupp/speedup_store

Speeding up store by removing nested query
gamesh411 · Oct 8, 2024 · a6663b3 · a6663b3
2 parents 86ce67a + 7a21ed1
commit a6663b3
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/web/server/codechecker_server/api/mass_store_run.py b/web/server/codechecker_server/api/mass_store_run.py
@@ -1319,11 +1319,12 @@ def get_skip_handler(
             .join(DBReport, DBReport.bug_id == ReviewStatusRule.bug_hash) \
             .filter(sqlalchemy.and_(
                 DBReport.run_id == run_id,
-                DBReport.review_status_is_in_source.is_(False),
-                ReviewStatusRule.bug_hash.in_(self.__new_report_hashes)))
+                DBReport.review_status_is_in_source.is_(False)))
 
         # Set the newly stored reports
         for review_status, db_report in reports_to_rs_rules:
+            if db_report.bug_id not in self.__new_report_hashes:
+                continue
             old_report = None
             if db_report.bug_id in report_to_report_id:
                 old_report = report_to_report_id[db_report.bug_id][0]