~ubuntu-branches/ubuntu/natty/postgresql-8.4/natty-updates

« back to all changes in this revision

Viewing changes to src/backend/commands/tablecmds.c

  • Committer: Package Import Robot
  • Author(s): Martin Pitt
  • Date: 2011-12-15 10:56:20 UTC
  • mfrom: (1.3.4)
  • Revision ID: package-import@ubuntu.com-20111215105620-tiz2wt392wn7ihnl
Tags: 8.4.10-0ubuntu0.11.04
* New upstream release (LP: #904631):
  - Fix bugs in information_schema.referential_constraints view.
    This view was being insufficiently careful about matching the
    foreign-key constraint to the depended-on primary or unique key
    constraint. That could result in failure to show a foreign key
    constraint at all, or showing it multiple times, or claiming that
    it depends on a different constraint than the one it really does.
    Since the view definition is installed by initdb, merely upgrading
    will not fix the problem. If you need to fix this in an existing
    installation, you can (as a superuser) drop the information_schema
    schema then re-create it by sourcing
    "SHAREDIR/information_schema.sql". (Run pg_config --sharedir if
    you're uncertain where "SHAREDIR" is.) This must be repeated in
    each database to be fixed.
  - Fix incorrect replay of WAL records for GIN index updates.
    This could result in transiently failing to find index entries
    after a crash, or on a hot-standby server. The problem would be
    repaired by the next "VACUUM" of the index, however.
  - Fix TOAST-related data corruption during CREATE TABLE dest AS
    SELECT - FROM src or INSERT INTO dest SELECT * FROM src.
    If a table has been modified by "ALTER TABLE ADD COLUMN", attempts
    to copy its data verbatim to another table could produce corrupt
    results in certain corner cases. The problem can only manifest in
    this precise form in 8.4 and later, but we patched earlier versions
    as well in case there are other code paths that could trigger the
    same bug.
  - Fix race condition during toast table access from stale syscache
    entries.
  - Track dependencies of functions on items used in parameter default
    expressions. Previously, a referenced object could be dropped without
    having dropped or modified the function, leading to misbehavior when the
    function was used. Note that merely installing this update will not fix
    the missing dependency entries; to do that, you'd need to "CREATE OR
    REPLACE" each such function afterwards. If you have functions whose
    defaults depend on non-built-in objects, doing so is recommended.
  - Allow inlining of set-returning SQL functions with multiple OUT
    parameters.
  - Make DatumGetInetP() unpack inet datums that have a 1-byte header,
    and add a new macro, DatumGetInetPP(), that does not.
  - Improve locale support in money type's input and output.
    Aside from not supporting all standard lc_monetary formatting
    options, the input and output functions were inconsistent, meaning
    there were locales in which dumped money values could not be
    re-read.
  - Don't let transform_null_equals affect CASE foo WHEN NULL ...
    constructs. transform_null_equals is only supposed to affect foo = NULL
    expressions written directly by the user, not equality checks
    generated internally by this form of CASE.
  - Change foreign-key trigger creation order to better support
    self-referential foreign keys. For a cascading foreign key that
    references its own table, a row update will fire both the ON UPDATE
    trigger and the CHECK trigger as one event. The ON UPDATE trigger must
    execute first, else the CHECK will check a non-final state of the row
    and possibly throw an inappropriate error. However, the firing order of
    these triggers is determined by their names, which generally sort in
    creation order since the triggers have auto-generated names following
    the convention "RI_ConstraintTrigger_NNNN". A proper fix would require
    modifying that convention, which we will do in 9.2, but it seems risky
    to change it in existing releases. So this patch just changes the
    creation order of the triggers. Users encountering this type of error
    should drop and re-create the foreign key constraint to get its triggers
    into the right order.
  - Avoid floating-point underflow while tracking buffer allocation
    rate.
  - Preserve blank lines within commands in psql's command history.
    The former behavior could cause problems if an empty line was
    removed from within a string literal, for example.
  - Fix pg_dump to dump user-defined casts between auto-generated
    types, such as table rowtypes.
  - Use the preferred version of xsubpp to build PL/Perl, not
    necessarily the operating system's main copy.
  - Fix incorrect coding in "contrib/dict_int" and "contrib/dict_xsyn".
  - Honor query cancel interrupts promptly in pgstatindex().
  - Ensure VPATH builds properly install all server header files.
  - Shorten file names reported in verbose error messages.
    Regular builds have always reported just the name of the C file
    containing the error message call, but VPATH builds formerly
    reported an absolute path name.

Show diffs side-by-side

added added

removed removed

Lines of Context:
5267
5267
        CommandCounterIncrement();
5268
5268
 
5269
5269
        /*
5270
 
         * Build and execute a CREATE CONSTRAINT TRIGGER statement for the CHECK
5271
 
         * action for both INSERTs and UPDATEs on the referencing table.
5272
 
         */
5273
 
        CreateFKCheckTrigger(myRel, fkconstraint, constraintOid, true);
5274
 
        CreateFKCheckTrigger(myRel, fkconstraint, constraintOid, false);
5275
 
 
5276
 
        /*
5277
5270
         * Build and execute a CREATE CONSTRAINT TRIGGER statement for the ON
5278
5271
         * DELETE action on the referenced table.
5279
5272
         */
5371
5364
        fk_trigger->args = NIL;
5372
5365
 
5373
5366
        (void) CreateTrigger(fk_trigger, constraintOid, false);
 
5367
 
 
5368
        /* Make changes-so-far visible */
 
5369
        CommandCounterIncrement();
 
5370
 
 
5371
        /*
 
5372
         * Build and execute CREATE CONSTRAINT TRIGGER statements for the CHECK
 
5373
         * action for both INSERTs and UPDATEs on the referencing table.
 
5374
         *
 
5375
         * Note: for a self-referential FK (referencing and referenced tables are
 
5376
         * the same), it is important that the ON UPDATE action fires before the
 
5377
         * CHECK action, since both triggers will fire on the same row during an
 
5378
         * UPDATE event; otherwise the CHECK trigger will be checking a non-final
 
5379
         * state of the row.  Because triggers fire in name order, we are
 
5380
         * effectively relying on the OIDs of the triggers to sort correctly as
 
5381
         * text.  This will work except when the OID counter wraps around or adds
 
5382
         * a digit, eg "99999" sorts after "100000".  That is infrequent enough,
 
5383
         * and the use of self-referential FKs is rare enough, that we live with
 
5384
         * it for now.  There will be a real fix in PG 9.2.
 
5385
         */
 
5386
        CreateFKCheckTrigger(myRel, fkconstraint, constraintOid, true);
 
5387
        CreateFKCheckTrigger(myRel, fkconstraint, constraintOid, false);
5374
5388
}
5375
5389
 
5376
5390
/*