Re: [ pg_ctl ] Review Request for Adding Pre-check User Script Feature - Mailing list pgsql-hackers

From Kisoon Kwon
Subject Re: [ pg_ctl ] Review Request for Adding Pre-check User Script Feature
Date
Msg-id CAGOrKoWtegzW+ZhzKANDZX_qorv3isHqTdX6BW1y19-WTqvU8A@mail.gmail.com
Whole thread Raw
In response to [ pg_ctl ] Review Request for Adding Pre-check User Script Feature  (김명준 <rayjun0525@gmail.com>)
Responses Re: [ pg_ctl ] Review Request for Adding Pre-check User Script Feature
List pgsql-hackers
Hi,

0. For more understanding, can you give me an example about your patch?
1. Instead of using chmod itself, it would be better to use chmod_recursive().
2. It needs to follow the invent convention - it includes 4 spaces now.

Thank you,

Kisoon Kwon

2024년 7월 16일 (화) 오후 3:40, 김명준 <rayjun0525@gmail.com>님이 작성:

Hello,

I have been considering adding a user script that performs pre-checks before executing the start, stop, and restart operations in pg_ctl. I believe it is necessary for pg_ctl to support an extension that can prevent various issues that might occur when using start and stop. To this end, I have sought a way for users to define and use their own logic. The existing behavior remains unchanged, and the feature can be used optionally when needed.

The verification of the code was carried out using the methods described below, and I would like to request additional opinions or feedback. Tests were conducted using make check and through direct testing under various scenarios. As this is my first contribution, there might be aspects I missed or incorrectly designed.

I would appreciate it if you could review this.

Thank you.


Myoungjun Kim / South Korea

pgsql-hackers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: Injection points: preloading and runtime arguments
Next
From: jian he
Date:
Subject: Re: altering a column's collation leaves an invalid foreign key