Change reliability to minimum for hash join nodes that were terminated early

Alexandra Pervushina · Alexandra Pervushina · commit c84dc34f1e26 · 2023-03-19T22:11:38.000Z
diff --git a/postprocessing.c b/postprocessing.c
@@ -336,6 +336,17 @@ should_learn(PlanState *ps, AQOPlanNode *node, aqo_obj_stat *ctx,
 	}
 	else if (ctx->learn)
 	{
+		elog(NOTICE, "fss %d,  IsNull %s, nloops %lf", node->fss, TupIsNull(ps->ps_ResultTupleSlot) ? "true" : "false", ps->instrument->nloops);
+		if (IsA(ps, HashJoinState))
+		{
+			HashJoinState *hjstate = castNode(HashJoinState, ps);
+			elog(NOTICE, "TerminatedEarly %s", hjstate->hj_TerminatedEarly ? "true" : "false");
+			if (hjstate->hj_TerminatedEarly)
+			{
+				*rfactor = RELIABILITY_MIN;
+				return true;
+			}
+		}
 		*rfactor = RELIABILITY_MAX;
 		return true;
 	}

Original file line number	Diff line number	Diff line change
`@@ -336,6 +336,17 @@ should_learn(PlanState ps, AQOPlanNode node, aqo_obj_stat *ctx,`
`336`	`336`	`}`
`337`	`337`	`else if (ctx->learn)`
`338`	`338`	`{`
	`339`	`+ elog(NOTICE, "fss %d, IsNull %s, nloops %lf", node->fss, TupIsNull(ps->ps_ResultTupleSlot) ? "true" : "false", ps->instrument->nloops);`
	`340`	`+ if (IsA(ps, HashJoinState))`
	`341`	`+ {`
	`342`	`+ HashJoinState *hjstate = castNode(HashJoinState, ps);`
	`343`	`+ elog(NOTICE, "TerminatedEarly %s", hjstate->hj_TerminatedEarly ? "true" : "false");`
	`344`	`+ if (hjstate->hj_TerminatedEarly)`
	`345`	`+ {`
	`346`	`+ *rfactor = RELIABILITY_MIN;`
	`347`	`+ return true;`
	`348`	`+ }`
	`349`	`+ }`
`339`	`350`	`*rfactor = RELIABILITY_MAX;`
`340`	`351`	`return true;`
`341`	`352`	`}`