Re: Re[2]: [PATCH] Change simple_heap_insert() to a macro

2018-10-15 Thread Tom Lane
=?UTF-8?B?QW5kcmV5IEtseWNoa292?= writes: >>  simple_heap_insert() is used in catalog updates and such. Does that have >>  any measurable performance impact? > I guess this change doesn't give us an incredible performance increase except > there will no redundant function call. > I don't see any

Re[2]: [PATCH] Change simple_heap_insert() to a macro

2018-10-12 Thread Andrey Klychkov
> simple_heap_insert() is used in catalog updates and such. Does that have > any measurable performance impact? I guess this change doesn't give us an incredible performance increase except there will no redundant function call. I don't see any reasons against to use the proposed macro instead of