NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite899742

committed

Improve stability of identity.sql regression test.

I noticed while trying to run the regression tests under a lowgeqo_threshold that one query on information_schema.columns hadunstable (as in, variable from one run to the next) output order.This is pretty unsurprising given the complexity of the underlyingplan. Interestingly, of this test's three nigh-identical queries oninformation_schema.columns, the other two already had ORDER BY clausesguaranteeing stable output. Let's make this one look the same.Back-patch to v10 where this test was added. We've not heard fieldreports of the test failing, but this experience shows that it canhappen when testing under even slightly unusual conditions.

1 parent10c5291 commite899742Copy full SHA for e899742

File tree

2 files changed

-2

lines changed

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

2 files changed

-2

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -335,7 +335,7 @@ SELECT * FROM itest6;`
`335`	`335`	`102 \|`
`336`	`336`	`(3 rows)`
`337`	`337`
`338`		`-SELECT table_name, column_name, is_identity, identity_generation FROM information_schema.columns WHERE table_name = 'itest6';`
	`338`	`+SELECT table_name, column_name, is_identity, identity_generation FROM information_schema.columns WHERE table_name = 'itest6' ORDER BY 1, 2;`
`339`	`339`	`table_name \| column_name \| is_identity \| identity_generation`
`340`	`340`	`------------+-------------+-------------+---------------------`
`341`	`341`	`itest6 \| a \| YES \| BY DEFAULT`

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ INSERT INTO itest6 DEFAULT VALUES;`
`208`	`208`	`INSERT INTO itest6 DEFAULTVALUES;`
`209`	`209`	`SELECT*FROM itest6;`
`210`	`210`
`211`		`-SELECT table_name, column_name, is_identity, identity_generationFROMinformation_schema.columnsWHERE table_name='itest6';`
	`211`	`+SELECT table_name, column_name, is_identity, identity_generationFROMinformation_schema.columnsWHERE table_name='itest6'ORDER BY1,2;`
`212`	`212`
`213`	`213`	`ALTERTABLE itest6 ALTER COLUMN bSET INCREMENT BY2;-- fail, not identity`
`214`	`214`

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!

Commite899742

File tree

2 files changed

2 files changed

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

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

0 commit comments