Re: [PATCH] elimination of code duplication in DefineOpFamily() - Mailing list pgsql-hackers

From Brent Dombrowski
Subject Re: [PATCH] elimination of code duplication in DefineOpFamily()
Date
Msg-id 4C197AEA.3030108@gmail.com
Whole thread Raw
In response to [PATCH] elimination of code duplication in DefineOpFamily()  (KaiGai Kohei <kaigai@ak.jp.nec.com>)
Responses Re: [PATCH] elimination of code duplication in DefineOpFamily()  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
I have not been able to find any comments or discussion on this patch.

Contents and Purpose:
================
This patch removes duplicate code in opclasscmds.c. It removes the 
duplicate code from DefineOpFamily by calling CreateOpFamily.

No new regression test or documentation are included with the patch. 
Since it is not adding, removing, or changing any functionality, they 
are most likely not necessary. The function call parameters remain the same.

Initial Run
========
The patch applied cleanly to HEAD. The regression tests pass both before 
and after the patch.

Performance
=========
No changes in performance were observed. I do not have an appropriate 
setup to properly test for performance impacts. Since this changes a 
define function, I would not expect it to be frequently used.

Conclusion
========
The patch eliminates duplicate code as expected.

Brent Dombrowski




pgsql-hackers by date:

Previous
From: "Joshua D. Drake"
Date:
Subject: Re: New PGXN Extension site
Next
From: Robert Haas
Date:
Subject: Re: InvalidXLogRecPtr in docs