On 16/01/2019 17:39, David G. Johnston wrote:
> On Wednesday, January 16, 2019, Peter Eisentraut <[email protected]
> <mailto:[email protected]>> wrote:
> 
>     Increase test coverage in RI_FKey_pk_upd_check_required()
> 
>     This checks the case where the primary key has at least one null
>     column.
> 
> 
> Sorry if this seems pedantic but you are testing where a unique index
> has at least one null column value.  A primary key, per the docs, cannot
> contain null in any of its columns.  I read the commit message and
> thought maybe that had changed while I wasn’t looking...

Yeah, it's the foreign key referencing the primary key in this case.

You are correct, the message is a bit wrong.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Reply via email to