Review: Patch insert throw error when year field len > 4 for timestamptz datatype

Started by Adrian Klaverabout 12 years ago3 messages
#1Adrian Klaver
adrian.klaver@gmail.com

Initial review of the patch submitted in this message and listed in the
current CommitFest:

/messages/by-id/CAGPqQf3XwWC_4fhiNz_G6EcvPs_OV3k2pe4-aJ1dg4iyY+f_Dw@mail.gmail.com

This patch would seem to be already committed here

http://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=7778ddc7a2d5b006edbfa69cdb44b8d8c24ec1ff

Is a review necessary at this point?

--
Adrian Klaver
adrian.klaver@gmail.com

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#2Peter Geoghegan
pg@heroku.com
In reply to: Adrian Klaver (#1)
Re: Review: Patch insert throw error when year field len > 4 for timestamptz datatype

On Thu, Nov 14, 2013 at 4:53 PM, Adrian Klaver <adrian.klaver@gmail.com> wrote:

Is a review necessary at this point?

No. Just mark it committed.

--
Peter Geoghegan

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Bruce Momjian
bruce@momjian.us
In reply to: Peter Geoghegan (#2)
Re: Review: Patch insert throw error when year field len > 4 for timestamptz datatype

On Thu, Nov 14, 2013 at 04:58:55PM -0800, Peter Geoghegan wrote:

On Thu, Nov 14, 2013 at 4:53 PM, Adrian Klaver <adrian.klaver@gmail.com> wrote:

Is a review necessary at this point?

No. Just mark it committed.

Oh, sorry, I didn't realize it was in the commit-fest app.

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

+ Everyone has their own god. +

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers