Hi -
About to push this little ditty.
commit a448a4d7997b9d7949a0040c4958eb84bf5e8e0f (HEAD -> master)
Author: Frank Ch. Eigler <f...@redhat.com>
Date: Thu Apr 15 06:29:01 2021 -0400
debuginfod: only update database stats once per groom
On very large servers, each database-stat counting pass can take tens
of minutes (!), and doing it twice per groom pass does not seriously
improve data quality. Just do it once, after stale data removal &
basic sqlite vacuum.
Signed-off-by: Frank Ch. Eigler <f...@redhat.com>
diff --git a/debuginfod/ChangeLog b/debuginfod/ChangeLog
index 3bd2ff606aa6..ed2f77cfaece 100644
--- a/debuginfod/ChangeLog
+++ b/debuginfod/ChangeLog
@@ -1,3 +1,7 @@
+2021-04-15 Frank Ch. Eigler <f...@redhat.com>
+
+ * debuginfod.cxx (groom): Only update database stats once.
+
2021-04-15 Frank Ch. Eigler <f...@redhat.com>
* debuginfod.cxx (elf_classify): Recognize symtab-only stripped files
diff --git a/debuginfod/debuginfod.cxx b/debuginfod/debuginfod.cxx
index 2d73a136ae5e..50777f1fa193 100644
--- a/debuginfod/debuginfod.cxx
+++ b/debuginfod/debuginfod.cxx
@@ -3100,8 +3100,6 @@ void groom()
struct timespec ts_start, ts_end;
clock_gettime (CLOCK_MONOTONIC, &ts_start);
- database_stats_report();
-
// scan for files that have disappeared
sqlite_ps files (db, "check old files", "select s.mtime, s.file, f.name from
"
BUILDIDS "_file_mtime_scanned s, " BUILDIDS "_files f "
- FChE
