Re: [PATCH] replace float8 with int in date2isoweek() and date2isoyear()

2025-07-13 Thread Sergey Fukanchik
Thank you! -- Sergey On 7/13/25 18:39, Tom Lane wrote: Sergey Fukanchik writes: I'm attaching v2 of the patch with "result" renamed to "week". I pushed it already, actually. regards, tom lane

Re: [PATCH] replace float8 with int in date2isoweek() and date2isoyear()

2025-07-13 Thread Tom Lane
Sergey Fukanchik writes: > I'm attaching v2 of the patch with "result" renamed to "week". I pushed it already, actually. regards, tom lane

Re: [PATCH] replace float8 with int in date2isoweek() and date2isoyear()

2025-07-13 Thread Sergey Fukanchik
Sure, I'm attaching v2 of the patch with "result" renamed to "week". -- Sergey On 7/12/25 18:15, Tom Lane wrote: =?utf-8?q?=D0=A4=D1=83=D0=BA=D0=B0=D0=BD=D1=87=D0=B8=D0=BA_=D0=A1=D0=B5=D1=80=D0=B3=D0=B5=D0=B9?= writes: Hi PG hackers, I found suspicious use of float8 in date2isoweek() and d

Re: [PATCH] replace float8 with int in date2isoweek() and date2isoyear()

2025-07-12 Thread Tom Lane
=?utf-8?q?=D0=A4=D1=83=D0=BA=D0=B0=D0=BD=D1=87=D0=B8=D0=BA_=D0=A1=D0=B5=D1=80=D0=B3=D0=B5=D0=B9?= writes: > Hi PG hackers, > I found suspicious use of float8 in date2isoweek() and date2isoyear(). In both > cases float8 is only used for storing the value, while the entire calculation > on the righ

[PATCH] replace float8 with int in date2isoweek() and date2isoyear()

2025-07-12 Thread Фуканчик Сергей
k', timestamp with time zone '2004-02-29 15:44:17.71393' ) AS week_trunc; --- Sergey From e3aac9f2ca12b60ff35d01ea69e4ffbaeea30656 Mon Sep 17 00:00:00 2001 From: Sergey Fukanchik Date: Sat, 12 Jul 2025 13:25:18 +0300 Subject: [PATCH] Replace float8 with int in date2isoweek and date2isoyear