| From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Hongxu Ma <interma(at)outlook(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: PSQL error: total cell count of XXX exceeded |
| Date: | 2023-11-21 16:14:22 |
| Message-ID: | 202311211614.3oweambmaqs4@alvherre.pgsql |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2023-Nov-21, Tom Lane wrote:
> Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> > Right. I made a few more adjustments, including the additional overflow
> > check in printTableInit that Tom Lane suggested, and pushed this.
>
> Committed patch LGTM.
Thanks for looking!
> > It's a bit annoying that the error recovery decision of this code is to
> > exit the process with an error. [...]
> > TBH though, I've never hit that code in real usage.
>
> Yeah, I think the reason it's stayed like that for 25 years is that
> nobody's hit the case in practice. Changing the API would be a bit
> troublesome, too, because we don't know if anybody besides psql
> uses it.
True.
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2023-11-21 16:41:09 | Re: Add recovery to pg_control and remove backup_label |
| Previous Message | Alvaro Herrera | 2023-11-21 16:13:12 | Re: Remove distprep |