On Tue, 20 Jan 2026 at 08:01, Hannu Krosing <[email protected]> wrote: > I changed the last open-ended chunk to use ctid >= (N,1) for clarity > but did not change anything else.
You have: int max_table_segment_pages; /* chunk when relpages is above this */ and: opts->max_table_segment_pages = UINT32_MAX; /* == InvalidBlockNumber, disable chunking by default */ It's not valid to assign UINT32_MAX to a signed int. > To me it looked like having a loop around the whole thing when there > is no chunking would complicate things for anyone reading the code. The problem I have with it is the duplicate code. If you don't want to loop around the standard code, then make a function and call that instead of copying and pasting the code. I'd also get rid of the "chunking" boolean and make use of InvalidBlockNumber to determine if the range is constrained. It also seems very strange that you opted to do that just for endPage and not for startPage. > > 4. I think using "int" here is a future complaint waiting to happen. > > > > + if (!option_parse_int(optarg, "--huge-table-chunk-pages", 1, INT32_MAX, > > + &dopt.huge_table_chunk_pages)) > > > > I bet we'll eventually see a complaint that someone can't make the > > segment size larger than 16TB. I think option_parse_uint32() might be > > called for. > > I have not yet done anything with this yet, so the maximum chunk size > for now is half of the maximum relpages. OK. I can look again once all that's done. David
