MIME-Version: 1.0
References: <CAJTYsWWNfF+hMpbsTDF8NMr+AuqoDfNxR=oHUZm7xqGP+dJ9rA@mail.gmail.com>
 <aenQ37wBgKqZb7a8@paquier.xyz> <FE1D12FF-7E5D-4ED0-A59F-BEA6DEF7EE2C@yesql.se>
 <1444609.1776958814@sss.pgh.pa.us>
In-Reply-To: <1444609.1776958814@sss.pgh.pa.us>
From: Ayush Tiwari <ayushtiwari.slg01@gmail.com>
Date: Thu, 23 Apr 2026 22:06:41 +0530
Message-ID: <CAJTYsWUJFHhqiGD+6vw6m0ELqjajoKev2UutFUMB2uQDjm6+TA@mail.gmail.com>
Subject: Re: to_date()/to_timestamp() silently accept month=0 and day=0
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Daniel Gustafsson <daniel@yesql.se>, Michael Paquier <michael@paquier.xyz>, 
	pgsql-bugs@lists.postgresql.org
Content-Type: multipart/alternative; boundary="0000000000007ef638065023410c"
Archived-At: <https://www.postgresql.org/message-id/CAJTYsWUJFHhqiGD%2B6vw6m0ELqjajoKev2UutFUMB2uQDjm6%2BTA%40mail.gmail.com>
Precedence: bulk

--0000000000007ef638065023410c
Content-Type: text/plain; charset="UTF-8"

Hi,

On Thu, 23 Apr 2026 at 21:10, Tom Lane <tgl@sss.pgh.pa.us> wrote:

>
> Looking at the code, I think it intentionally interprets zero as
> "missing data".  See for example the stanza at formatting.c:4650ff
> where tm_mon and tm_mday can be backfilled from a DDD field.
>
> I'm disinclined to change the behavior around this; you're far
> more likely to get complaints than kudos.
>
>
If we are not going to change the behaviour, imo, we should
atleast add a test case to depict this scenario and comment
that says this is expected behaviour according to the community.

Regards,
Ayush

--0000000000007ef638065023410c
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Hi,</div><br><div class=3D"gmail_quote gmail_quote_co=
ntainer"><div dir=3D"ltr" class=3D"gmail_attr">On Thu, 23 Apr 2026 at 21:10=
, Tom Lane &lt;<a href=3D"mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>&g=
t; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0p=
x 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Looking at the code, I think it intentionally interprets zero as<br>
&quot;missing data&quot;.=C2=A0 See for example the stanza at formatting.c:=
4650ff<br>
where tm_mon and tm_mday can be backfilled from a DDD field.<br>
<br>
I&#39;m disinclined to change the behavior around this; you&#39;re far<br>
more likely to get complaints than kudos.<br><br></blockquote><div><br></di=
v><div>If we are not going to change the behaviour, imo, we should</div><di=
v>atleast=C2=A0add a test case to depict this scenario and comment</div><di=
v>that says this is expected behaviour according to the community.=C2=A0</d=
iv><div><br></div><div>Regards,<br>Ayush</div></div></div>

--0000000000007ef638065023410c--