On 2020-01-24 01:21, Thomas Munro wrote:
On Fri, Jan 24, 2020 at 11:12 AM Tom Lane <t...@sss.pgh.pa.us> wrote:
I happened to notice this comment in the logic in
ATAddForeignKeyConstraint that tries to decide if it can skip
revalidating a foreign-key constraint after a DDL change:
* Since we require that all collations share the same notion of
* equality (which they do, because texteq reduces to bitwise
* equality), we don't compare collation here.
Hasn't this been broken by the introduction of nondeterministic
collations?
Similar words appear in the comment for ri_GenerateQualCollation().
The calls to this function are all conditional on
!get_collation_isdeterministic(). The comment should perhaps be changed.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
