NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit0ad41cf

committed

Fix unique INCLUDE indexes on partitioned tables

We were considering the INCLUDE columns as part of the key, allowingunicity-violating rows to be inserted in different partitions.Concurrent development conflict ineb7ed3f and8224de4.Reported-by: Justin PryzbyDiscussion:https://postgr.es/m/20190109065109.GA4285@telsasoft.com

1 parentbb24439 commit0ad41cfCopy full SHA for 0ad41cf

File tree

3 files changed

-1

lines changed

src
- backend/commands
  - indexcmds.c
- test/regress
  - expected
    - indexing.out
  - sql
    - indexing.sql

3 files changed

-1

lines changed

`‎src/backend/commands/indexcmds.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -721,7 +721,7 @@ DefineIndex(Oid relationId,`
`721`	`721`	`errdetail("%s constraints cannot be used when partition keys include expressions.",`
`722`	`722`	`constraint_type)));`
`723`	`723`
`724`		`-for (j=0;j<indexInfo->ii_NumIndexAttrs;j++)`
	`724`	`+for (j=0;j<indexInfo->ii_NumIndexKeyAttrs;j++)`
`725`	`725`	`{`
`726`	`726`	`if (key->partattrs[i]==indexInfo->ii_IndexAttrNumbers[j])`
`727`	`727`	`{`

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

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1411,3 +1411,6 @@ insert into covidxpart values (4, 1);`
`1411`	`1411`	`insert into covidxpart values (4, 1);`
`1412`	`1412`	`ERROR: duplicate key value violates unique constraint "covidxpart4_a_b_idx"`
`1413`	`1413`	`DETAIL: Key (a)=(4) already exists.`
	`1414`	`+create unique index on covidxpart (b) include (a); -- should fail`
	`1415`	`+ERROR: insufficient columns in UNIQUE constraint definition`
	`1416`	`+DETAIL: UNIQUE constraint on table "covidxpart" lacks column "a" which is part of the partition key.`

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

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -756,3 +756,4 @@ create unique index on covidxpart4 (a);`
`756`	`756`	`altertable covidxpart attach partition covidxpart4 forvaluesin (4);`
`757`	`757`	`insert into covidxpartvalues (4,1);`
`758`	`758`	`insert into covidxpartvalues (4,1);`
	`759`	`+createunique indexon covidxpart (b) include (a);-- should fail`

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!

Commit0ad41cf

File tree

3 files changed

3 files changed

`‎src/backend/commands/indexcmds.c`

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

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

0 commit comments