Re: [PATCH] Do not use StdRdOptions in Access Methods

Nikolay Shaplov <dhyan@nataraj.su>

From: Nikolay Shaplov <dhyan@nataraj.su>
To: Dent John <denty@qqdd.eu>
Cc: Thomas Munro <thomas.munro@gmail.com>, Tomas Vondra <tomas.vondra@2ndquadrant.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, "Iwata, Aya" <iwata.aya@jp.fujitsu.com>, Alvaro Herrera <alvherre@2ndquadrant.com>, Dmitry Dolgov <9erthalion6@gmail.com>, Michael Paquier <michael@paquier.xyz>
Date: 2019-10-09T21:37:44Z
Lists: pgsql-hackers
В письме от среда, 9 октября 2019 г. 20:26:14 MSK пользователь Dent John 
написал:

> I didn’t spot it was an existing pattern.
Sorry, this might be my mistake I should explicitly mention it in the first 
place. 
 
> And I agree — making the code uniform will make it easier to evolve in
> future.
> 
> Gets my vote.
Thanks!

Can you please check, I did not do any mistake while copying and adapting the 
code.

-- 
Software Developer: https://www.upwork.com/freelancers/~014a87e140ff02c0da
Body-oriented Therapist: https://vk.com/nataraj_rebalancing  (Russian)

Commits

  1. Refactor reloption handling for index AMs in-core

  2. Cleanup code in reloptions.h regarding reloption handling

  3. Refactor code building relation options

  4. Add some assertions to view reloption macros