On Fri, Jan 30, 2015 at 10:32 AM, Tom Lane <t...@sss.pgh.pa.us> wrote: > [blah] > What I did about this was to leave the behavior alone in back branches, > but insist on a type match in HEAD. I think we can reasonably tighten > the type requirements in a new major release, but doing it in minor > releases is probably a bad idea. Hm. OK. I am fine with that for the back branches. Thanks for tightening things on master as well.
> * I thought it was odd to throw an error for NULL input, especially > an "infinite recursion" error. However, even with your patch the code > would've dumped core on a null current_key value (since it would've > passed a null start_with down to the next recursion level). What I > changed it to was to omit the recursion test (and hence print the row) > and then not recurse at a null. This seems consistent and reasonable. This sounds reasonable to me as well. > * I made a few other minor cleanups as well, in particular getting > rid of some unnecessary pstrdup() steps. Thanks! -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers