Re: tuplesort_gettuple_common() and *should_free argument
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Peter Geoghegan <pg@heroku.com>
Cc: Heikki Linnakangas <hlinnaka@iki.fi>,
Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2016-12-08T21:02:05Z
Lists: pgsql-hackers
Peter Geoghegan <pg@heroku.com> writes: > On Wed, Dec 7, 2016 at 11:55 PM, Heikki Linnakangas <hlinnaka@iki.fi> wrote: >> Should we be worried about breaking the API of tuplesort_get* functions? >> They might be used by extensions. I think that's OK, but wanted to bring it >> up. This would be only for master, of course, and any breakage would be >> straightforward to fix. > I don't think so. I'm not aware of any third party extensions that > call tuplesort.c routines, despite having looked for them. I noticed > that pg_repack doesn't. For any that do, they'll break in a > predictable, obvious way. Adding or subtracting function arguments is something we do all the time. As long as it's not proposed for back-patch, I don't see a problem. regards, tom lane
Commits
-
Fix duplicated words in comment.
- b58c433ef90b 10.0 landed
-
Remove should_free arguments to tuplesort routines.
- 3856cf9607f4 10.0 landed
-
Fix use-after-free around DISTINCT transition function calls.
- d8589946ddd5 10.0 cited