Commit083b913

committed

On second thought, explain why date_trunc("week") on interval values is

not supported in the error message, rather than the docs.

1 parent74f4881 commit083b913Copy full SHA for 083b913

File tree

+11

-9

lines changed

+11

-9

lines changed

Lines changed: 0 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -7050,11 +7050,6 @@ date_trunc('<replaceable>field</replaceable>', <replaceable>source</replaceable>`
`7050`	`7050`	`</simplelist>`
`7051`	`7051`	`</para>`
`7052`	`7052`
`7053`		`- <para>`
`7054`		`- <literal>week</literal> is not supported for <type>interval</>`
`7055`		`- values because months usually have fractional weeks.`
`7056`		`- </para>`
`7057`		`-`
`7058`	`7053`	`<para>`
`7059`	`7054`	`Examples:`
`7060`	`7055`	`<screen>`

Lines changed: 11 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -3710,10 +3710,17 @@ interval_trunc(PG_FUNCTION_ARGS)`
`3710`	`3710`	`break;`
`3711`	`3711`
`3712`	`3712`	`default:`
`3713`		`-ereport(ERROR,`
`3714`		`-(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),`
`3715`		`-errmsg("interval units \"%s\" not supported",`
`3716`		`-lowunits)));`
	`3713`	`+if (val==DTK_WEEK)`
	`3714`	`+ereport(ERROR,`
	`3715`	`+(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),`
	`3716`	`+errmsg("interval units \"%s\" not supported "`
	`3717`	`+"because months usually have fractional weeks",`
	`3718`	`+lowunits)));`
	`3719`	`+else`
	`3720`	`+ereport(ERROR,`
	`3721`	`+(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),`
	`3722`	`+errmsg("interval units \"%s\" not supported",`
	`3723`	`+lowunits)));`
`3717`	`3724`	`}`
`3718`	`3725`
`3719`	`3726`	`if (tm2interval(tm,fsec,result)!=0)`

Comments

(0)