On Thu, Jan 9, 2025 at 12:15 PM jian he <jian.universal...@gmail.com> wrote: > hi. > in create_grouping_expr_infos > > tce = lookup_type_cache(exprType((Node *) tle->expr), > TYPECACHE_BTREE_OPFAMILY); > if (!OidIsValid(tce->btree_opf) || > !OidIsValid(tce->btree_opintype)) > return; > .... > /* > * Get the operator in the btree's opfamily. > */ > eq_op = get_opfamily_member(tce->btree_opf, > tce->btree_opintype, > tce->btree_opintype, > BTEqualStrategyNumber); > if (!OidIsValid(eq_op)) > return; > eq_opfamilies = get_mergejoin_opfamilies(eq_op); > if (!eq_opfamilies) > return; > btree_opfamily = linitial_oid(eq_opfamilies); > > > If eq_op is valid, then we don't need to call get_mergejoin_opfamilies? > since get_mergejoin_opfamilies output will be the same as tce->btree_opf. > and we already checked (tce->btree_opf) is valid. > > In other words, I think eq_op is valid imply > that tce->btree_opf is the value (btree opfamily) we need.
Nice catch! Actually, we can use tce->btree_opf directly, without needing to check its equality operator, since we know it's a btree opfamily and it's valid. If it were a different opfamily (such as a hash opfamily), we would need to look up its equality operator, and select some btree opfamily that that operator is part of. But in this case, that's not necessary. Thanks Richard
