| From: | jian he <jian(dot)universality(at)gmail(dot)com> |
|---|---|
| To: | Corey Huinker <corey(dot)huinker(at)gmail(dot)com> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: statatt_build_stavalues->LOCAL_FCINFO wrong number |
| Date: | 2026-06-30 04:20:57 |
| Message-ID: | CACJufxHEq6MLbaqK22oUGU-776=11Cf_awRbrAD47bHjxDOf=Q@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi.
I have one more tiny question...
pg_restore_attribute_stats->attribute_statistics_update->statatt_build_stavalues->FunctionCallInvoke
which may fail, causing pg_restore_attribute_stats to return false.
/ * Parameters corresponding to ANYARRAY columns are instead passed in as text
* values, which is a valid input string for an array of the type or element
* type of the attribute. Any error generated by the array_in() function will
* in turn fail the function.
*/
To me, the phrase "fail the function" suggests that it raises
ereport(ERROR), rather than simply returning false.
However, here, array_in() does not appear to trigger a hard
ereport(ERROR). Instead, it causes attribute_statistics_update() to return
false.
Because of that, I'm not sure what the last sentence is trying to convey. Does
"fail the function" mean "cause the function to return false" rather than "raise
an error"?
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Feng Wu | 2026-06-30 04:26:32 | [PATCH] Avoid internal error for invalid interval typmods |
| Previous Message | Feng Wu | 2026-06-30 04:18:36 | [PATCH] Avoid internal error for invalid interval typmods |