From: Jelte Fennema <postgres@jeltef.nl> Sent: Monday, September 11, 2023 15:04 To: Hongxu Ma <interma@outlook.com> Cc: David G. Johnston <david.g.johnston@gmail.com>; PostgreSQL Hackers <pgsql-hackers@postgresql.org> Subject: Re: PSQL error: total cell count of XXX exceeded
On Mon, 11 Sept 2023 at 08:51, Hongxu Ma <interma@outlook.com> wrote: > > I created a patch to fix it. > Really appreciate to anyone can help to review it. > Thanks.
I think "product" as a variable name isn't very descriptive. Let's call it total_cells (or something similar instead).
Other than that I think it's a good change. content->cellsadded is also a long, So I agree that I don't think the limit of int cells was intended here.