Re: Add generate_series(date,date) and generate_series(date,date,integer)

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Christoph Berg <myon(at)debian(dot)org>, David Fetter <david(at)fetter(dot)org>, Torsten Zuehlsdorff <mailinglists(at)toco-domains(dot)de>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Add generate_series(date,date) and generate_series(date,date,integer)
Date: 2016-02-22 07:07:25
Message-ID: CAB7nPqRoFNJCvVCeuTQx3SE3P8w_2KjxouobYL7a8R+couY64Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Feb 22, 2016 at 6:52 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Oooh ... actually, that works today if you consider the SRF-in-targetlist
> case:
>
> regression=# select generate_series(now(), 'infinity', '1 day') limit 10;
> generate_series
> -------------------------------
> 2016-02-21 16:51:03.303064-05
> 2016-02-22 16:51:03.303064-05
> 2016-02-23 16:51:03.303064-05
> 2016-02-24 16:51:03.303064-05
> 2016-02-25 16:51:03.303064-05
> 2016-02-26 16:51:03.303064-05
> 2016-02-27 16:51:03.303064-05
> 2016-02-28 16:51:03.303064-05
> 2016-02-29 16:51:03.303064-05
> 2016-03-01 16:51:03.303064-05
> (10 rows)
>
> Time: 8.457 ms
>
> Given that counterexample, I think we not only shouldn't back-patch such a
> change but should reject it altogether.

Ouch, good point. The overflows are a different problem that we had
better address though (still on my own TODO list)...
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Craig Ringer 2016-02-22 07:24:23 Re: Writing new unit tests with PostgresNode
Previous Message Chapman Flack 2016-02-22 06:36:18 Re: about google summer of code 2016