Re: [HACKERS] Date conversion using day of week

On Fri, Apr  1, 2011 at 03:58:48AM +1100, Brendan Jurd wrote:
> On 1 April 2011 03:32, Adrian Klaver <adrian.klaver@gmail.com> wrote:
> > Now I am confused the docs say:
> >
> > D       day of the week, Sunday(1) to Saturday(7)
> > ID      ISO day of the week, Monday(1) to Sunday(7)
> >
> > This would seem to say they both are one-based but differ on the day that is
> > 1.
> 
> That's correct for the user-facing interpretation.  Internally,
> however, Gregorian day-of-week is represented with Sunday = 0.  I
> can't see any good reason in the code for why that should be so, but
> it was like that when I found it, and until now I haven't had any
> cause to mess with it.
> 
> My suggestion for moving forward basically still stands, though.  We'd
> need to standardise the use of TmFromChar.d to either one of the
> 1-based conventions, and convert to the other one as required in
> do_to_timestamp.  The Gregorian convention is probably the right
> choice for the standard, even though it has the week starting on a
> Sunday (ridiculous!) because it means less converting for the majority
> of cases.

Yes, I went in the direction you suggested with patch posted in recent
thread.

-- 
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +