NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf90c708

committed

Fix wrong units in two ExplainPropertyFloat calls.

This is only a latent bug, since these calls are only reached fornon-text output formats, and currently none of those will printthe units. Still, we should get it right in case that ever changes.Justin PryzbyDiscussion:https://postgr.es/m/20210415163846.GA3315@telsasoft.com

1 parent029c5ac commitf90c708Copy full SHA for f90c708

File tree

1 file changed

-2

lines changed

src/backend/commands
- explain.c

1 file changed

-2

lines changed

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

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1631,9 +1631,9 @@ ExplainNode(PlanState planstate, List ancestors,`
`1631`	`1631`	`{`
`1632`	`1632`	`if (es->timing)`
`1633`	`1633`	`{`
`1634`		`-ExplainPropertyFloat("Actual Startup Time","s",startup_ms,`
	`1634`	`+ExplainPropertyFloat("Actual Startup Time","ms",startup_ms,`
`1635`	`1635`	`3,es);`
`1636`		`-ExplainPropertyFloat("Actual Total Time","s",total_ms,`
	`1636`	`+ExplainPropertyFloat("Actual Total Time","ms",total_ms,`
`1637`	`1637`	`3,es);`
`1638`	`1638`	`}`
`1639`	`1639`	`ExplainPropertyFloat("Actual Rows",NULL,rows,0,es);`

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!

Commitf90c708

File tree

1 file changed

1 file changed

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

0 commit comments