Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-19 Thread Dagfinn Ilmari Mannsåker
Chao Li writes: > On Wed, Nov 19, 2025 at 4:33 PM Peter Eisentraut > wrote: > >> I find the data structures that you have constructed here barely >> understandable: >> >> my %required_by_type = ( >> int => [qw(min max)], >> real => [qw(min max)], >> enum => [qw(o

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-19 Thread Chao Li
On Wed, Nov 19, 2025 at 4:33 PM Peter Eisentraut wrote: > I find the data structures that you have constructed here barely > understandable: > > my %required_by_type = ( > int => [qw(min max)], > real => [qw(min max)], > enum => [qw(options)], > ); > > f

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-19 Thread Peter Eisentraut
I find the data structures that you have constructed here barely understandable: my %required_by_type = ( int => [qw(min max)], real => [qw(min max)], enum => [qw(options)], ); for my $f (@required_common, @{ $required_by_type{$entry->{type} // ''} // [] })

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-18 Thread Mahmoud Ayman
The following review has been posted through the commitfest application: make installcheck-world: not tested Implements feature: not tested Spec compliant: not tested Documentation:not tested I tested patch v2 on top of current master. - The patch applies cleanly. - F

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-17 Thread Chao Li
Added to CF: https://commitfest.postgresql.org/patch/6226/ > On Nov 12, 2025, at 09:10, Chao Li wrote: > > > > On Mon, Nov 10, 2025 at 8:02 PM Dagfinn Ilmari Mannsåker > wrote: > Chao Li writes: > > > Hi Hacker, > > > > While working on the other patch and editing guc_parameters.dat, I > >

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-11 Thread Chao Li
On Mon, Nov 10, 2025 at 8:02 PM Dagfinn Ilmari Mannsåker wrote: > Chao Li writes: > > > Hi Hacker, > > > > While working on the other patch and editing guc_parameters.dat, I > > mistakenly deleted a “max” value line, then I got the following error > > during build: > > > > ``` > > '/opt/homebrew

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-11 Thread Álvaro Herrera
On 2025-Nov-10, Chao Li wrote: > While working on the other patch and editing guc_parameters.dat, I > mistakenly deleted a “max” value line, then I got the following error > during build: > The error message is unclear and is not helpful to identify the real issue. Funny -- I made a very similar

Re: gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-10 Thread Dagfinn Ilmari Mannsåker
Chao Li writes: > Hi Hacker, > > While working on the other patch and editing guc_parameters.dat, I > mistakenly deleted a “max” value line, then I got the following error > during build: > > ``` > '/opt/homebrew/bin/perl' ../../../src/backend/utils/misc/gen_guc_tables.pl > ../../../src/backend/u

gen_guc_tables.pl: Validate required GUC fields before code generation

2025-11-10 Thread Chao Li
Hi Hacker, While working on the other patch and editing guc_parameters.dat, I mistakenly deleted a “max” value line, then I got the following error during build: ``` '/opt/homebrew/bin/perl' ../../../src/backend/utils/misc/gen_guc_tables.pl ../../../src/backend/utils/misc/guc_parameters.dat guc_t