NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit5c456d3

committed

Work around cross-version-upgrade issues created by commit 9e38c2bb5.

Summarily changing the STYPE of regression-test aggregates thatdepend on array_append or array_cat is an issue for the buildfarm'scross-version-upgrade tests, because those aggregates (as definedin the back branches) now won't load into HEAD. Although this seemslike only a minimal risk for genuine user-defined aggregates, weneed to do something for the buildfarm. Hence, adjust the aggregatedefinitions, in both HEAD and the back branches.Discussion:https://postgr.es/m/1401824.1604537031@sss.pgh.pa.usDiscussion:https://postgr.es/m/E1kaQ2c-0005lx-Eg@gemulon.postgresql.org

1 parent6daf725 commit5c456d3Copy full SHA for 5c456d3

File tree

2 files changed

+21

-16

lines changed

src/test/regress
- expected
  - polymorphism.out
- sql
  - polymorphism.sql

2 files changed

+21

-16

lines changed

`‎src/test/regress/expected/polymorphism.out`

Lines changed: 13 additions & 11 deletions

Original file line number	Diff line number	Diff line change
`@@ -729,24 +729,24 @@ select q2, sql_if(q2 > 0, q2, q2 + 1) from int8_tbl;`
`729`	`729`	`(5 rows)`
`730`	`730`
`731`	`731`	`-- another sort of polymorphic aggregate`
`732`		`-CREATE AGGREGATEarray_cat_accum (anyarray)`
	`732`	`+CREATE AGGREGATEarray_larger_accum (anyarray)`
`733`	`733`	`(`
`734`		`- sfunc =array_cat,`
	`734`	`+ sfunc =array_larger,`
`735`	`735`	`stype = anyarray,`
`736`	`736`	`initcond = '{}'`
`737`	`737`	`);`
`738`		`-SELECTarray_cat_accum(i)`
	`738`	`+SELECTarray_larger_accum(i)`
`739`	`739`	`FROM (VALUES (ARRAY[1,2]), (ARRAY[3,4])) as t(i);`
`740`		`-array_cat_accum`
`741`		`------------------`
`742`		`- {1,2,3,4}`
	`740`	`+array_larger_accum`
	`741`	`+--------------------`
	`742`	`+ {3,4}`
`743`	`743`	`(1 row)`
`744`	`744`
`745`		`-SELECTarray_cat_accum(i)`
	`745`	`+SELECTarray_larger_accum(i)`
`746`	`746`	`FROM (VALUES (ARRAY[row(1,2),row(3,4)]), (ARRAY[row(5,6),row(7,8)])) as t(i);`
`747`		`- array_cat_accum`
`748`		`------------------------------------`
`749`		`- {"(1,2)","(3,4)","(5,6)","(7,8)"}`
	`747`	`+array_larger_accum`
	`748`	`+--------------------`
	`749`	`+ {"(5,6)","(7,8)"}`
`750`	`750`	`(1 row)`
`751`	`751`
`752`	`752`	`-- another kind of polymorphic aggregate`
`@@ -786,6 +786,8 @@ create aggregate build_group(int8, integer) (`
`786`	`786`	`STYPE = int8[]`
`787`	`787`	`);`
`788`	`788`	`-- check proper resolution of data types for polymorphic transfn/finalfn`
	`789`	`+create function first_el_transfn(anyarray, anyelement) returns anyarray as`
	`790`	`+'select $1 \|\| $2' language sql immutable;`
`789`	`791`	`create function first_el(anyarray) returns anyelement as`
`790`	`792`	`'select $1[1]' language sql strict immutable;`
`791`	`793`	`create aggregate first_el_agg_f8(float8) (`
`@@ -794,7 +796,7 @@ create aggregate first_el_agg_f8(float8) (`
`794`	`796`	`FINALFUNC = first_el`
`795`	`797`	`);`
`796`	`798`	`create aggregate first_el_agg_any(anyelement) (`
`797`		`- SFUNC =array_append,`
	`799`	`+ SFUNC =first_el_transfn,`
`798`	`800`	`STYPE = anyarray,`
`799`	`801`	`FINALFUNC = first_el`
`800`	`802`	`);`

`‎src/test/regress/sql/polymorphism.sql`

Lines changed: 8 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -498,17 +498,17 @@ select q2, sql_if(q2 > 0, q2, q2 + 1) from int8_tbl;`
`498`	`498`
`499`	`499`	`-- another sort of polymorphic aggregate`
`500`	`500`
`501`		`-CREATEAGGREGATEarray_cat_accum (anyarray)`
	`501`	`+CREATEAGGREGATEarray_larger_accum (anyarray)`
`502`	`502`	`(`
`503`		`- sfunc=array_cat,`
	`503`	`+ sfunc=array_larger,`
`504`	`504`	`stype= anyarray,`
`505`	`505`	`initcond='{}'`
`506`	`506`	`);`
`507`	`507`
`508`		`-SELECTarray_cat_accum(i)`
	`508`	`+SELECTarray_larger_accum(i)`
`509`	`509`	`FROM (VALUES (ARRAY[1,2]), (ARRAY[3,4]))as t(i);`
`510`	`510`
`511`		`-SELECTarray_cat_accum(i)`
	`511`	`+SELECTarray_larger_accum(i)`
`512`	`512`	`FROM (VALUES (ARRAY[row(1,2),row(3,4)]), (ARRAY[row(5,6),row(7,8)]))as t(i);`
`513`	`513`
`514`	`514`	`-- another kind of polymorphic aggregate`
`@@ -549,6 +549,9 @@ create aggregate build_group(int8, integer) (`
`549`	`549`
`550`	`550`	`-- check proper resolution of data types for polymorphic transfn/finalfn`
`551`	`551`
	`552`	`+createfunctionfirst_el_transfn(anyarray, anyelement) returns anyarrayas`
	`553`	`+'select $1 \|\| $2' language sql immutable;`
	`554`	`+`
`552`	`555`	`createfunctionfirst_el(anyarray) returns anyelementas`
`553`	`556`	`'select $1[1]' language sql strict immutable;`
`554`	`557`
`@@ -559,7 +562,7 @@ create aggregate first_el_agg_f8(float8) (`
`559`	`562`	`);`
`560`	`563`
`561`	`564`	`createaggregatefirst_el_agg_any(anyelement) (`
`562`		`- SFUNC=array_append,`
	`565`	`+ SFUNC=first_el_transfn,`
`563`	`566`	`STYPE= anyarray,`
`564`	`567`	`FINALFUNC= first_el`
`565`	`568`	`);`

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!

Commit5c456d3

File tree

2 files changed

2 files changed

`‎src/test/regress/expected/polymorphism.out`

`‎src/test/regress/sql/polymorphism.sql`

0 commit comments