Analyze all databases with vacuumdb

NikolayS · vitabaks · commit 0de0536509bf · 2021-02-03T19:37:41.000Z
When using the "logical" databases initialization,
we need to analyze our target database (that one that
has the data we just loaded).

It looks like conn.DBName is pointing to "postgres"
when we run vacuumdb.

Switch to using "-all" for running ANALYZE. The most time is
to be spent in our target database since only that database has
significant volumes of data.
diff --git a/pkg/retrieval/engine/postgres/logical/logical.go b/pkg/retrieval/engine/postgres/logical/logical.go
@@ -15,7 +15,7 @@ func buildAnalyzeCommand(conn Connection, parallelJobs int) []string {
 		"--analyze",
 		"--jobs", strconv.Itoa(parallelJobs),
 		"--username", conn.Username,
-		"--dbname", conn.DBName,
+		"--all",
 	}
 
 	return analyzeCmd

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ func buildAnalyzeCommand(conn Connection, parallelJobs int) []string {`
`15`	`15`	`"--analyze",`
`16`	`16`	`"--jobs", strconv.Itoa(parallelJobs),`
`17`	`17`	`"--username", conn.Username,`
`18`		`- "--dbname", conn.DBName,`
	`18`	`+ "--all",`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`return analyzeCmd`