| From: | jian he <jian(dot)universality(at)gmail(dot)com> |
|---|---|
| To: | Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: SQL:2011 application time |
| Date: | 2023-09-12 09:01:36 |
| Message-ID: | CACJufxFxQH5NxD9UDvbcRCPtaSga+y0+Wwt54PA4KahLwhahOA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
hi. some trivial issue:
in src/backend/catalog/index.c
/* * System attributes are never null, so no need to check. */
if (attnum <= 0)
since you already checked attnum == 0
so here you can just attnum < 0?
-------------------------------------------------
ERROR: column "valid_at" named in WITHOUT OVERLAPS is not a range type
IMHO, "named" is unnecessary.
-------------------------------------------------
doc/src/sgml/catalogs.sgml
pg_constraint adds another attribute (column): contemporal, seems no doc entry.
also the temporal in oxford definition is "relating to time", here we
can deal with range.
So maybe "temporal" is not that accurate?
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dean Rasheed | 2023-09-12 09:09:17 | Re: Infinite Interval |
| Previous Message | Peter Smith | 2023-09-12 09:00:14 | Re: Add 'worker_type' to pg_stat_subscription |