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

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 26, 2016 at 7:53 AM, Michael Paquier <span
dir="ltr"><<ahref="mailto:michael.paquier@gmail.com" target="_blank">michael.paquier@gmail.com</a>></span>
wrote:<br/><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span
class="">OnTue, Jan 26, 2016 at 7:00 PM, Torsten Zuehlsdorff<br /> <<a
href="mailto:mailinglists@toco-domains.de">mailinglists@toco-domains.de</a>>wrote:<br /> ><br /> > On
26.01.201607:52, Simon Riggs wrote:<br /> ><br /> >>> Imagine for example a script that in some rare cases
passeshappens to<br /> >>> pass infinity into generate_series() - in that case I'd much rather error<br />
>>>out than wait till the end of the universe.<br /> >>><br /> >>> So +1 from me to checking
forinfinity.<br /> >>><br /> >><br /> >> +1<br /> >><br /> >> ERROR infinite result
setsare not supported, yet<br /> ><br /> ><br /> > Maybe we should skip the "yet". Or do we really plan to
supportthem in<br /> > (infinite) future? ;)<br /> ><br /> > +1 from me to check infinity also.<br /><br
/></span>Somethinglike the patch attached would be fine? This wins a backpatch<br /> because the query continuously
runningeats memory, no?<br /><span class="HOEnZb"><font color="#888888">--<br /> Michael<br /></font></span><br /><br
/>--<br /> Sent via pgsql-hackers mailing list (<a
href="mailto:pgsql-hackers@postgresql.org">pgsql-hackers@postgresql.org</a>)<br/> To make changes to your
subscription:<br/><a href="http://www.postgresql.org/mailpref/pgsql-hackers" rel="noreferrer"
target="_blank">http://www.postgresql.org/mailpref/pgsql-hackers</a><br/><br /></blockquote></div><br /></div><div
class="gmail_extra">I'llmodify my generate_series_date to give similar errors.</div><div class="gmail_extra"><br
/></div><divclass="gmail_extra">I have no opinion on backpatch.<br /><br />+1 for flippant references to
infinity.</div><divclass="gmail_extra"><br /></div></div>