Bugfix. Normalize cardinality error.

danolivo · danolivo · commit 5ba53add5244 · 2022-05-11T15:14:12.000+05:00
diff --git a/postprocessing.c b/postprocessing.c
@@ -497,7 +497,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
`@@ -497,7 +497,7 @@ learnOnPlanState(PlanState p, void context)`
`497`	`497`	`/* Exclude "not executed" nodes from error calculation to reduce fluctuations. */`
`498`	`498`	`if (!notExecuted)`
`499`	`499`	`{`
`500`		`- cardinality_sum_errors += fabs(predicted - learn_rows);`
	`500`	`+ cardinality_sum_errors += fabs(log(predicted) - log(learn_rows));`
`501`	`501`	`cardinality_num_objects += 1;`
`502`	`502`	`}`
`503`	`503`