diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2019-01-25 11:44:26 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2019-01-25 11:44:26 -0500 |
| commit | ff750ce2d82979e9588c629955e161a9379b05f3 (patch) | |
| tree | 006efc81e0cc0fb590cbd2d95e69f5800c5fe2bc /src | |
| parent | 6119060d01ab2dfb9e2c1c101595f2de7890965d (diff) | |
| download | postgresql-ff750ce2d82979e9588c629955e161a9379b05f3.tar.gz postgresql-ff750ce2d82979e9588c629955e161a9379b05f3.zip | |
Teach nulltestsel() that system columns are never NULL.
While it's perhaps unlikely that users would write an explicit test
like "ctid IS NULL", this function is also used in range estimation,
and an incorrect answer can throw off the results for tight ranges.
Anyway it's not much code so we might as well do it.
Edmund Horner
Discussion: https://postgr.es/m/CAMyN-kCa3BFUFrCTtQeprxTU1anCd3Pua7zXstGCKq4pXgjukw@mail.gmail.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/utils/adt/selfuncs.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/backend/utils/adt/selfuncs.c b/src/backend/utils/adt/selfuncs.c index dcb35d89752..4af8c80848d 100644 --- a/src/backend/utils/adt/selfuncs.c +++ b/src/backend/utils/adt/selfuncs.c @@ -1797,6 +1797,15 @@ nulltestsel(PlannerInfo *root, NullTestType nulltesttype, Node *arg, return (Selectivity) 0; /* keep compiler quiet */ } } + else if (vardata.var && IsA(vardata.var, Var) && + ((Var *) vardata.var)->varattno < 0) + { + /* + * There are no stats for system columns, but we know they are never + * NULL. + */ + selec = (nulltesttype == IS_NULL) ? 0.0 : 1.0; + } else { /* |