NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit292661e

committed

Fix unstable regression test added by commits59b71c6 et al.

The query didn't really have a preferred index, leading to platform-specific choices of which one to use. Adjust it to make sure tenk1_hundredis always chosen.Per buildfarm.

1 parentf16a095 commit292661eCopy full SHA for 292661e

File tree

2 files changed

-9

lines changed

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

2 files changed

-9

lines changed

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

Lines changed: 6 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -2003,7 +2003,7 @@ CREATE AGGREGATE balk(`
`2003`	`2003`	`STYPE = int8,`
`2004`	`2004`	`"PARALLEL" = SAFE,`
`2005`	`2005`	`INITCOND = '0');`
`2006`		`-SELECT balk(1) FROM tenk1;`
	`2006`	`+SELECT balk(hundred) FROM tenk1;`
`2007`	`2007`	`balk`
`2008`	`2008`	`------`
`2009`	`2009`
`@@ -2037,17 +2037,17 @@ CREATE AGGREGATE balk(`
`2037`	`2037`	`ALTER TABLE tenk1 set (parallel_workers = 4);`
`2038`	`2038`	`SET LOCAL parallel_setup_cost=0;`
`2039`	`2039`	`SET LOCAL max_parallel_workers_per_gather=4;`
`2040`		`-EXPLAIN (COSTS OFF) SELECT balk(1) FROM tenk1;`
`2041`		`-QUERY PLAN`
`2042`		`---------------------------------------------------------------------------------`
	`2040`	`+EXPLAIN (COSTS OFF) SELECT balk(hundred) FROM tenk1;`
	`2041`	`+ QUERY PLAN`
	`2042`	`+-------------------------------------------------------------------------`
`2043`	`2043`	`Finalize Aggregate`
`2044`	`2044`	`-> Gather`
`2045`	`2045`	`Workers Planned: 4`
`2046`	`2046`	`-> Partial Aggregate`
`2047`		`- -> Parallel Index Only Scan usingtenk1_thous_tenthous on tenk1`
	`2047`	`+ -> Parallel Index Only Scan usingtenk1_hundred on tenk1`
`2048`	`2048`	`(5 rows)`
`2049`	`2049`
`2050`		`-SELECT balk(1) FROM tenk1;`
	`2050`	`+SELECT balk(hundred) FROM tenk1;`
`2051`	`2051`	`balk`
`2052`	`2052`	`------`
`2053`	`2053`

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

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -862,7 +862,7 @@ CREATE AGGREGATE balk(`
`862`	`862`	`"PARALLEL"= SAFE,`
`863`	`863`	`INITCOND='0');`
`864`	`864`
`865`		`-SELECT balk(1)FROM tenk1;`
	`865`	`+SELECT balk(hundred)FROM tenk1;`
`866`	`866`
`867`	`867`	`ROLLBACK;`
`868`	`868`
`@@ -896,7 +896,7 @@ ALTER TABLE tenk1 set (parallel_workers = 4);`
`896`	`896`	`SET LOCAL parallel_setup_cost=0;`
`897`	`897`	`SET LOCAL max_parallel_workers_per_gather=4;`
`898`	`898`
`899`		`-EXPLAIN (COSTS OFF)SELECT balk(1)FROM tenk1;`
`900`		`-SELECT balk(1)FROM tenk1;`
	`899`	`+EXPLAIN (COSTS OFF)SELECT balk(hundred)FROM tenk1;`
	`900`	`+SELECT balk(hundred)FROM tenk1;`
`901`	`901`
`902`	`902`	`ROLLBACK;`

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!

Commit292661e

File tree

2 files changed

2 files changed

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

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

0 commit comments