only add unused edges if we have HLL

lnkuiper · Sep 18, 2024 · b8541f3 · b8541f3
1 parent 079f7a2
commit b8541f3
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/optimizer/join_order/cardinality_estimator.cpp b/src/optimizer/join_order/cardinality_estimator.cpp
@@ -296,8 +296,9 @@ DenomInfo CardinalityEstimator::GetDenominator(JoinRelationSet &set) {
 	for (auto &edge : edges) {
 		if (subgraphs.size() == 1 && subgraphs.at(0).relations->ToString() == set.ToString()) {
 			// the first subgraph has connected all the desired relations, just skip the rest of the edges
-			auto edge_tdom = edge.has_tdom_hll ? edge.tdom_hll : edge.tdom_no_hll;
-			unused_edge_tdoms.insert(edge_tdom);
+			if (edge.has_tdom_hll) {
+				unused_edge_tdoms.insert(edge.tdom_hll);
+			}
 			continue;
 		}