NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit63e2004

committed

Fix assertion failure for REFRESH MATERIALIZED VIEW in PL.

This was due to incomplete implementation of rowcount reportingfor RMV, which was due to initial waffling on whether it shouldbe provided. It seems unlikely to be a useful or universallyavailable number as more sophisticated techniques for maintainingmatviews are added, so remove the partial support rather thancompleting it.Per report of Jeevan Chalke, but with a different fix

1 parent2317a63 commit63e2004Copy full SHA for 63e2004

File tree

1 file changed

-7

lines changed

src/backend/executor
- spi.c

1 file changed

-7

lines changed

`‎src/backend/executor/spi.c`

Lines changed: 0 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -2122,13 +2122,6 @@ _SPI_execute_plan(SPIPlanPtr plan, ParamListInfo paramLI,`
`2122`	`2122`	`if (((CreateTableAsStmt*)stmt)->is_select_into)`
`2123`	`2123`	`res=SPI_OK_SELINTO;`
`2124`	`2124`	`}`
`2125`		`-elseif (IsA(stmt,RefreshMatViewStmt))`
`2126`		`-{`
`2127`		`-Assert(strncmp(completionTag,`
`2128`		`-"REFRESH MATERIALIZED VIEW ",23)==0);`
`2129`		`-_SPI_current->processed=strtoul(completionTag+23,`
`2130`		`-NULL,10);`
`2131`		`-}`
`2132`	`2125`	`elseif (IsA(stmt,CopyStmt))`
`2133`	`2126`	`{`
`2134`	`2127`	`Assert(strncmp(completionTag,"COPY ",5)==0);`

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!

Commit63e2004

File tree

1 file changed

1 file changed

`‎src/backend/executor/spi.c`

0 commit comments