NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitcc4dd2a

committed

neqjoinsel must now pass through collation to eqjoinsel.

Since commit044c99b, eqjoinsel passes the passed-in collationto any operators it invokes. However, neqjoinsel failed to passon whatever collation it got, so that if we invoked acollation-dependent operator via that code path, we'd get "could notdetermine which collation to use for string comparison" or the like.Per report from Justin Pryzby. Back-patch to v12, like the previouscommit.Discussion:https://postgr.es/m/20200721191606.GL5748@telsasoft.com

1 parentac25e7b commitcc4dd2aCopy full SHA for cc4dd2a

File tree

1 file changed

-6

lines changed

src/backend/utils/adt
- selfuncs.c

1 file changed

-6

lines changed

`‎src/backend/utils/adt/selfuncs.c`

Lines changed: 9 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -2775,6 +2775,7 @@ neqjoinsel(PG_FUNCTION_ARGS)`
`2775`	`2775`	`Listargs= (List)PG_GETARG_POINTER(2);`
`2776`	`2776`	`JoinTypejointype= (JoinType)PG_GETARG_INT16(3);`
`2777`	`2777`	`SpecialJoinInfosjinfo= (SpecialJoinInfo)PG_GETARG_POINTER(4);`
	`2778`	`+Oidcollation=PG_GET_COLLATION();`
`2778`	`2779`	`float8result;`
`2779`	`2780`
`2780`	`2781`	`if (jointype==JOIN_SEMI\|\|jointype==JOIN_ANTI)`
`@@ -2821,12 +2822,14 @@ neqjoinsel(PG_FUNCTION_ARGS)`
`2821`	`2822`
`2822`	`2823`	`if (eqop)`
`2823`	`2824`	`{`
`2824`		`-result=DatumGetFloat8(DirectFunctionCall5(eqjoinsel,`
`2825`		`-PointerGetDatum(root),`
`2826`		`-ObjectIdGetDatum(eqop),`
`2827`		`-PointerGetDatum(args),`
`2828`		`-Int16GetDatum(jointype),`
`2829`		`-PointerGetDatum(sjinfo)));`
	`2825`	`+result=`
	`2826`	`+DatumGetFloat8(DirectFunctionCall5Coll(eqjoinsel,`
	`2827`	`+collation,`
	`2828`	`+PointerGetDatum(root),`
	`2829`	`+ObjectIdGetDatum(eqop),`
	`2830`	`+PointerGetDatum(args),`
	`2831`	`+Int16GetDatum(jointype),`
	`2832`	`+PointerGetDatum(sjinfo)));`
`2830`	`2833`	`}`
`2831`	`2834`	`else`
`2832`	`2835`	`{`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitcc4dd2a

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/selfuncs.c`

0 commit comments