Hi,
On 5/5/21 11:09 PM, Justin Pryzby wrote:
> Per sqlsmith.
>
> postgres=# SELECT pg_get_statisticsobjdef_expressions(123);
> ERROR: cache lookup failed for statistics object 123
> postgres=# \errverbose
> ERROR: XX000: cache lookup failed for statistics object 123
> LOCATION: pg_get_statisticsobjdef_expressions, ruleutils.c:1762
>
> The expectation is that sql callable functions should return null rather than
> hitting elog().
>
Right, thanks for noticing this.
> In the 003 patch, I wonder if this part should be updated, too:
>
> | ... which can greatly improve query plans that use the expression index.
>
> It can improve queries even that don't use the index, right ?
>
> Say, if a query has f(x) = 11, and the MCV list for the expression shows that
> 50% of the table has f(x)=11, then the query might decide to *not* use an index
> scan.
Yeah, it should talk about improving estimates, it's mostly unrelated to
using indexes.
regards
