Commit611019f

committed

pathman: opfamily lookup fix

1 parent1e82dc7 commit611019fCopy full SHA for 611019f

File tree

-7

lines changed

-7

lines changed

Lines changed: 3 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -398,8 +398,7 @@ load_check_constraints(Oid parent_oid, Snapshot snapshot)`
`398`	`398`	`boolbyVal=rangerel->by_val;`
`399`	`399`
`400`	`400`	`/* Sort ascending */`
`401`		`-tce=lookup_type_cache(prel->atttype,`
`402`		`-TYPECACHE_CMP_PROC \|TYPECACHE_CMP_PROC_FINFO);`
	`401`	`+tce=lookup_type_cache(prel->atttype,TYPECACHE_CMP_PROC \|TYPECACHE_CMP_PROC_FINFO);`
`403`	`402`	`qsort_type_cmp_func=&tce->cmp_proc_finfo;`
`404`	`403`	`globalByVal=byVal;`
`405`	`404`	`qsort(ranges,proc,sizeof(RangeEntry),cmp_range_entries);`
`@@ -458,7 +457,7 @@ validate_range_constraint(Expr expr, PartRelationInfo prel, Datum *min, Datum`
`458`	`457`	`if (!and_clause((Node*)expr))`
`459`	`458`	`return false;`
`460`	`459`
`461`		`-tce=lookup_type_cache(prel->atttype,TYPECACHE_EQ_OPR \|TYPECACHE_LT_OPR \|TYPECACHE_GT_OPR);`
	`460`	`+tce=lookup_type_cache(prel->atttype,TYPECACHE_BTREE_OPFAMILY);`
`462`	`461`
`463`	`462`	`/* check that left operand is >= operator */`
`464`	`463`	`opexpr= (OpExpr*)linitial(boolexpr->args);`
`@@ -517,7 +516,7 @@ validate_hash_constraint(Expr expr, PartRelationInfo prel, int *hash)`
`517`	`516`	`eqexpr= (OpExpr*)expr;`
`518`	`517`
`519`	`518`	`/* Is this an equality operator? */`
`520`		`-tce=lookup_type_cache(prel->atttype,TYPECACHE_EQ_OPR);`
	`519`	`+tce=lookup_type_cache(prel->atttype,TYPECACHE_BTREE_OPFAMILY);`
`521`	`520`	`if (get_op_opfamily_strategy(eqexpr->opno,tce->btree_opf)!=BTEqualStrategyNumber)`
`522`	`521`	`return false;`
`523`	`522`

Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -223,8 +223,7 @@ get_cmp_func(Oid type1, Oid type2)`
`223`	`223`
`224`	`224`	`cmp_func=palloc(sizeof(FmgrInfo));`
`225`	`225`	`tce=lookup_type_cache(type1,`
`226`		`-TYPECACHE_EQ_OPR \|TYPECACHE_LT_OPR \|TYPECACHE_GT_OPR \|`
`227`		`-TYPECACHE_CMP_PROC \|TYPECACHE_CMP_PROC_FINFO);`
	`226`	`+TYPECACHE_BTREE_OPFAMILY \|TYPECACHE_CMP_PROC \|TYPECACHE_CMP_PROC_FINFO);`
`228`	`227`	`cmp_proc_oid=get_opfamily_proc(tce->btree_opf,`
`229`	`228`	`type1,`
`230`	`229`	`type2,`
`@@ -1019,7 +1018,7 @@ handle_binary_opexpr(const PartRelationInfo prel, WrapperNode result,`
`1019`	`1018`
`1020`	`1019`	`/* Determine operator type */`
`1021`	`1020`	`tce=lookup_type_cache(v->vartype,`
`1022`		`-TYPECACHE_EQ_OPR \|TYPECACHE_LT_OPR \|TYPECACHE_GT_OPR \|TYPECACHE_CMP_PROC \|TYPECACHE_CMP_PROC_FINFO);`
	`1021`	`+TYPECACHE_BTREE_OPFAMILY \|TYPECACHE_CMP_PROC \|TYPECACHE_CMP_PROC_FINFO);`
`1023`	`1022`	`strategy=get_op_opfamily_strategy(expr->opno,tce->btree_opf);`
`1024`	`1023`	`cmp_proc_oid=get_opfamily_proc(tce->btree_opf,`
`1025`	`1024`	`c->consttype,`

Comments

(0)