public inbox for [email protected]  
help / color / mirror / Atom feed
From: Bruce Momjian <[email protected]>
To: [email protected]
To: [email protected]
Subject: Re: Argument 'week' not supported in date_trunc function with intervals
Date: Fri, 17 Jan 2020 21:20:51 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

On Fri, Jan  3, 2020 at 12:31:27PM +0000, PG Doc comments form wrote:
> The following documentation comment has been logged on the website:
> 
> Page: https://www.postgresql.org/docs/12/functions-datetime.html
> Description:
> 
> In the documentation of the date_trunc function (9.9.2), the first argument
> (field) cannot be 'week' if the second argument (source) is an interval:
> 
> csauer=# SELECT DATE_TRUNC('week', INTERVAL '7 days');                      
>                                                                             
>                                                                             
>                                                     
> ERROR:  interval units "week" not supported because months usually have
> fractional weeks
> csauer=# SELECT DATE_TRUNC('week', INTERVAL '1 week');                      
>                                                                             
>                                                                             
>                                                    
> ERROR:  interval units "week" not supported because months usually have
> fractional weeks
> 
> The documentation should state that explicitly.
> 
> I also think the error message could be improved, because in the cases above
> there are no months involved and the result should be 1.

Uh, we could modify the code so that we allow SELECT DATE_TRUNC('week',
INTERVAL '') to work if there is no month component, since interval is
made up of months, days, and time.  However, this would mean the
function would work with some interval values, and not others.  Is that
an improvement?

-- 
  Bruce Momjian  <[email protected]>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

+ As you are, so once was I.  As I am, so you will be. +
+                      Ancient Roman grave inscription +






view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Argument 'week' not supported in date_trunc function with intervals
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox