Bugfix. Normalize cardinality error.

danolivo · danolivo · commit c538b70cb7b8 · 2022-05-11T15:15:06.000+05:00
diff --git a/postprocessing.c b/postprocessing.c
@@ -435,7 +435,7 @@ learnOnPlanState(PlanState *p, void *context)
 	/* Exclude "not executed" nodes from error calculation to reduce fluctuations. */
 	if (!notExecuted)
 	{
-		cardinality_sum_errors += fabs(predicted - learn_rows);
+		cardinality_sum_errors += fabs(log(predicted) - log(learn_rows));
 		cardinality_num_objects += 1;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -435,7 +435,7 @@ learnOnPlanState(PlanState p, void context)`
`435`	`435`	`/* Exclude "not executed" nodes from error calculation to reduce fluctuations. */`
`436`	`436`	`if (!notExecuted)`
`437`	`437`	`{`
`438`		`- cardinality_sum_errors += fabs(predicted - learn_rows);`
	`438`	`+ cardinality_sum_errors += fabs(log(predicted) - log(learn_rows));`
`439`	`439`	`cardinality_num_objects += 1;`
`440`	`440`	`}`
`441`	`441`