Re: SQL:2011 application time

From: Paul Jungwirth <pj(at)illuminatedcomputing(dot)com>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: jian he <jian(dot)universality(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, vignesh C <vignesh21(at)gmail(dot)com>
Subject: Re: SQL:2011 application time
Date: 2025-02-26 05:15:43
Message-ID: 1085d8a3-17a7-4639-afd2-bd6dfa09713d@illuminatedcomputing.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2/21/25 07:21, Peter Eisentraut wrote:
> On 17.02.25 07:42, Paul Jungwirth wrote:
> I think maybe we have a different idea of what RESTRICT should do in the first place. Because all
> the different behavior options come from the same underlying difference.
>
> Consider a related example. What if you have in the referenced table just one row:
>
> (1, '[2000-01-01,2015-01-01)')
>
> and in the referencing row as above
>
> (1, '[2010-03-01,2010-04-01)')
>
> with ON UPDATE RESTRICT and ON DELETE RESTRICT. And then you run
>
> UPDATE pk SET valid_at = '[2000-01-01,2021-01-01)' WHERE id = 1;
>
> So this extends the valid_at of the primary key row, which is completely harmless for the
> referential integrity. But I argue that this is an error under ON UPDATE RESTRICT. Because that's
> the whole point of RESTRICT over NO ACTION: Even harmless changes to the primary key row are
> disallowed if the row is referenced.
>
> If we accept that this is an error, then the rest follows. If the primary row is split into two:
>
> (1, '[2000-01-01,2011-01-01)')
> (1, '[2011-01-01,2015-01-01)')
>
> then the command that extends the validity
>
> UPDATE pk SET valid_at = '[2011-01-01,2021-01-01)'
> WHERE id = 1 AND valid_at = '[2011-01-01,2015-01-01)';
>
> must also be an error, even though the row it is updating is not actually the one that is
> referenced. If this were allowed, then the behavior would be different depending on in which way
> the primary key ranges are split up, which is not what we want.

I agree with that last principle: it shouldn't matter how the primary keys are split up. But it
seems to me that "matches" in the standard should include the period. It does for NO ACTION, so why
not RESTRICT? That's why your example of expanding the referenced range succeeds. None of the
referenced moments were changed, so there are no referencing moments to match.

> I'm not sure what other behavior of RESTRICT there might be that is internally consistent and is
> meaningfully different from NO ACTION.

The difference between RESTRICT and NO ACTION for temporal foreign keys is the same as the
difference for ordinary foreign keys: we perform the check prior to applying any "action" or
allowing any other commands to provide substitutes for the lost references. There are tests in
sql/without_overlaps.sql showing how their behavior differs.

Also you haven't yet explained why anyone would *want* to use RESTRICT as you've described it, since
the temporal part of the key is just ignored, and they could just define a non-temporal foreign key
instead. Or to be precise, it fails *more* than a non-temporal foreign key, because changing the
period can violate the constraint, even though we ignore the period when looking for matches.

But since we don't agree on the behavior, it seems best to me to wait to implement RESTRICT. Not
much is lost, since NO ACTION is so similar. We can wait for the SQL committee to clarify things, or
see what another RDBMS vendor does.

FWIW IBM DB2 claims to support temporal RESTRICT foreign keys,[1] but this week I tested 11.5 and
12.1 via evaluation downloads, IBM Cloud, and AWS Marketplace. In all cases I got an error like this:

db2 => create table t (id integer not null, ds date not null, de date not null, name varchar(4000),
period business_time (ds, de));
DB20000I The SQL command completed successfully.
db2 => alter table t add constraint tpk primary key (id, business_time without overlaps);
DB20000I The SQL command completed successfully.
db2 =>
db2 => create table t2 (id integer not null, ds date not null, de date not null, name
varchar(4000), t_id integer, period business_time (ds, de));
DB20000I The SQL command completed successfully.
db2 => alter table t2 add constraint t2pk primary key (id, business_time without overlaps);
DB20000I The SQL command completed successfully.
db2 => alter table t2 add constraint t2fkt foreign key (t_id, period business_time) references t
(id, period business_time) on delete restrict;
DB21034E The command was processed as an SQL statement because it was not a
valid Command Line Processor command. During SQL processing it returned:
SQL0104N An unexpected token "business_time" was found following "gn key
(t_id, period". Expected tokens may include: "<space>". SQLSTATE=42601

It looks like the docs are just wrong, and they don't recognize the `period` keyword yet. (The error
message suggests that `period` is being interpreted as a column name, and there should be a comma or
closing paren after it.) I tried a lot of other guesses at different syntax, but nothing worked.
Maybe it is only supported on z/OS, not Linux? If anyone knows someone who works on/with DB2, I'd be
glad to talk to them.

Curiously, their docs say that temporal foreign keys *only* support ON DELETE RESTRICT:[2]

> ON DELETE RESTRICT must be specified when PERIOD BUSINESS_TIME is also specified.

Here are some patches removing support for RESTRICT and also rebasing to fix a lot of merge
conflicts. The rebase is to 6c349d83b6.

[1] https://www.ibm.com/docs/en/db2-for-zos/13?topic=constraints-referential
[2] https://www.ibm.com/docs/en/db2-for-zos/13?topic=statements-alter-table

Yours,

--
Paul ~{:-)
pj(at)illuminatedcomputing(dot)com

Attachment Content-Type Size
v49-0001-Remove-support-for-temporal-RESTRICT-foreign-key.patch text/x-patch 35.4 KB
v49-0002-Add-without_portion-GiST-support-proc.patch text/x-patch 40.9 KB
v49-0003-Add-UPDATE-DELETE-FOR-PORTION-OF.patch text/x-patch 192.2 KB
v49-0004-Add-CASCADE-SET-NULL-SET-DEFAULT-for-temporal-fo.patch text/x-patch 205.1 KB
v49-0005-Expose-FOR-PORTION-OF-to-plpgsql-triggers.patch text/x-patch 13.2 KB
v49-0006-Add-PERIODs.patch text/x-patch 547.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message John Naylor 2025-02-26 05:22:25 Re: Improve CRC32C performance on SSE4.2
Previous Message Tom Lane 2025-02-26 05:05:44 Re: Statistics Import and Export