On 2017-02-15 17:27:11 Robert Haas wrote: > On Tue, Feb 14, 2017 at 1:38 PM, Jim Nasby > <Jim(dot)Nasby(at)bluetreble(dot)com> wrote: > > On 2/14/17 3:13 AM, Seki, Eiji wrote: > >> +extern TransactionId GetOldestXmin(Relation rel, uint8 ignoreFlags); > > > > > > My impression is that most other places that do this sort of thing just call > > the argument 'flags', so as not to "lock in" a single idea of what the flags > > are for. I can't readily think of another use for flags in GetOldestXmin, > > but ISTM it's better to just go with "flags" instead of "ignoreFlags". > > I agree; also, many years ago a guy named Tom Lane told me that flags > argument should typically be declared as type "int". I've followed > that advice ever since.
Thank you for your comments. I reflected these comments to the attached patch. And I renamed IGNORE_XXX flags to PROCARRAY_XXX flags. -- Regards, Eiji Seki Fujitsu
get_oldest_xmin_with_ignore_flags_v2.patch
Description: get_oldest_xmin_with_ignore_flags_v2.patch
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers