From: Bruce Momjian Date: Wed, 25 Jun 2003 01:11:09 +0000 (+0000) Subject: - Corrections for tables, columns with uppercase characters X-Git-Tag: REL7_4_BETA1~334 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=51d16423ae2c8ecbf0005408b8137ce6bdbe148a;p=postgresql - Corrections for tables, columns with uppercase characters - Don't attempt to convert partial or expressional unique indexes - Don't attempt to convert unique indexes based on a non-default opclasses - Untested prevention of conversion of non-btree indexes unique indexes. Untested as postgresql doesn't allow hash, gist, or rtree based indexes to be unique. rbt=# create unique index t on a using hash (col); ERROR: DefineIndex: access method "hash" does not support UNIQUE indexes rbt=# create unique index t on a using gist (col); ERROR: DefineIndex: access method "gist" does not support UNIQUE indexes rbt=# select version(); version ------------------------------------------------------------------------ PostgreSQL 7.4devel on i386-unknown-freebsd4.8, compiled by GCC 2.95.4 Rod Taylor --- diff --git a/contrib/adddepend/README.adddepend b/contrib/adddepend/README.adddepend index f12c647561..d00689f2e5 100644 --- a/contrib/adddepend/README.adddepend +++ b/contrib/adddepend/README.adddepend @@ -16,9 +16,9 @@ individually. created but there was no indication that the index was created as a UNIQUE column constraint. -Fortunately, 7.3 now tracks such dependencies and handles these cases. -Unfortunately, PostgreSQL dumps from prior releases don't contain -such dependency information. +Fortunately, PostgreSQL 7.3 and later now tracks such dependencies +and handles these cases. Unfortunately, PostgreSQL dumps from prior +releases don't contain such dependency information. This script operates on >= 7.3 databases and adds dependency information for the objects listed above. It prompts the user on whether to create @@ -42,4 +42,4 @@ Options: the conversion for each item found. This forces YES to all questions. -Rod Taylor +Rod Taylor diff --git a/contrib/adddepend/adddepend b/contrib/adddepend/adddepend index 4fa8bc5d38..ea0c12690e 100755 --- a/contrib/adddepend/adddepend +++ b/contrib/adddepend/adddepend @@ -1,8 +1,8 @@ #!/usr/bin/perl -# $Id: adddepend,v 1.4 2003/05/14 03:25:55 tgl Exp $ +# $Id: adddepend,v 1.5 2003/06/25 01:11:09 momjian Exp $ # Project exists to assist PostgreSQL users with their structural upgrade -# from 7.2 (or prior) to 7.3 (possibly later). Must be run against a 7.3 +# from PostgreSQL 7.2 (or prior) to 7.3 or 7.4. Must be run against a 7.3 or 7.4 # database system (dump, upgrade daemon, restore, run this script) # # - Replace old style Foreign Keys with new style @@ -125,6 +125,20 @@ my $dbh = DBI->connect($dsn, $dbuser, $dbpass); # We want to control commits $dbh->{'AutoCommit'} = 0; +# PostgreSQL's version is used to determine what queries are required +# to retrieve a given information set. +my $sql_GetVersion = qq{ + SELECT cast(substr(version(), 12, 1) as integer) * 10000 + + cast(substr(version(), 14, 1) as integer) * 100 + as version; +}; + +my $sth_GetVersion = $dbh->prepare($sql_GetVersion); +$sth_GetVersion->execute(); +my $version = $sth_GetVersion->fetchrow_hashref; +my $pgversion = $version->{'version'}; + + # control where things get created my $sql = qq{ SET search_path = public; @@ -210,13 +224,13 @@ sub findForeignKeys } # Start off our column lists - my $key_cols = "$lcolumn_name"; - my $ref_cols = "$fcolumn_name"; + my $key_cols = "\"$lcolumn_name\""; + my $ref_cols = "\"$fcolumn_name\""; # Perhaps there is more than a single column while ($lcolumn_name = shift(@junk) and $fcolumn_name = shift(@junk)) { - $key_cols .= ", $lcolumn_name"; - $ref_cols .= ", $fcolumn_name"; + $key_cols .= ", \"$lcolumn_name\""; + $ref_cols .= ", \"$fcolumn_name\""; } my $trigsql = qq{ @@ -252,7 +266,7 @@ sub findForeignKeys /^RI_FKey_noaction_upd$/ && do {$updatetype = "ON UPDATE NO ACTION"; last;}; } - $triglist .= " DROP TRIGGER \"$trigname\" ON $tablename;\n"; + $triglist .= " DROP TRIGGER \"$trigname\" ON \"$tablename\";\n"; } @@ -264,8 +278,8 @@ sub findForeignKeys my $fkey = qq{ $triglist - ALTER TABLE $table ADD $constraint FOREIGN KEY ($key_cols) - REFERENCES $ftable($ref_cols) $matchtype $updatetype $deletetype; + ALTER TABLE \"$table\" ADD $constraint FOREIGN KEY ($key_cols) + REFERENCES \"$ftable\"($ref_cols) $matchtype $updatetype $deletetype; }; # Does the user want to upgrade this sequence? @@ -291,26 +305,67 @@ MSG # the corresponding entry in pg_constraint) sub findUniqueConstraints { - my $sql = qq{ - SELECT ci.relname AS index_name - , ct.relname AS table_name - , pg_catalog.pg_get_indexdef(indexrelid) AS constraint_definition - FROM pg_class AS ci - JOIN pg_index ON (ci.oid = indexrelid) - JOIN pg_class AS ct ON (ct.oid = indrelid) - JOIN pg_catalog.pg_namespace ON (ct.relnamespace = pg_namespace.oid) - WHERE indisunique - AND NOT EXISTS (SELECT TRUE - FROM pg_catalog.pg_depend - JOIN pg_catalog.pg_constraint ON (refobjid = pg_constraint.oid) - WHERE objid = indexrelid - AND objsubid = 0) - AND nspname NOT IN ('pg_catalog', 'pg_toast'); + my $sql; + if ( $pgversion >= 70400 ) { + $sql = qq{ + SELECT pg_index.*, quote_ident(ci.relname) AS index_name + , quote_ident(ct.relname) AS table_name + , pg_catalog.pg_get_indexdef(indexrelid) AS constraint_definition + , indclass + FROM pg_catalog.pg_class AS ci + JOIN pg_catalog.pg_index ON (ci.oid = indexrelid) + JOIN pg_catalog.pg_class AS ct ON (ct.oid = indrelid) + JOIN pg_catalog.pg_namespace ON (ct.relnamespace = pg_namespace.oid) + WHERE indisunique -- Unique indexes only + AND indpred IS NULL -- No Partial Indexes + AND indexprs IS NULL -- No expressional indexes + AND NOT EXISTS (SELECT TRUE + FROM pg_catalog.pg_depend + JOIN pg_catalog.pg_constraint + ON (refobjid = pg_constraint.oid) + WHERE objid = indexrelid + AND objsubid = 0) + AND nspname NOT IN ('pg_catalog', 'pg_toast'); + }; + } + else + { + $sql = qq{ + SELECT pg_index.*, quote_ident(ci.relname) AS index_name + , quote_ident(ct.relname) AS table_name + , pg_catalog.pg_get_indexdef(indexrelid) AS constraint_definition + , indclass + FROM pg_catalog.pg_class AS ci + JOIN pg_catalog.pg_index ON (ci.oid = indexrelid) + JOIN pg_catalog.pg_class AS ct ON (ct.oid = indrelid) + JOIN pg_catalog.pg_namespace ON (ct.relnamespace = pg_namespace.oid) + WHERE indisunique -- Unique indexes only + AND indpred = '' -- No Partial Indexes + AND indproc = 0 -- No expressional indexes + AND NOT EXISTS (SELECT TRUE + FROM pg_catalog.pg_depend + JOIN pg_catalog.pg_constraint + ON (refobjid = pg_constraint.oid) + WHERE objid = indexrelid + AND objsubid = 0) + AND nspname NOT IN ('pg_catalog', 'pg_toast'); + }; + } + + my $opclass_sql = qq{ + SELECT TRUE + FROM pg_catalog.pg_opclass + JOIN pg_catalog.pg_am ON (opcamid = pg_am.oid) + WHERE amname = 'btree' + AND pg_opclass.oid = ? + AND pg_opclass.oid < 15000; }; my $sth = $dbh->prepare($sql) || triggerError($!); + my $opclass_sth = $dbh->prepare($opclass_sql) || triggerError($!); $sth->execute(); +ITERATION: while (my $row = $sth->fetchrow_hashref) { # Fetch vars @@ -318,6 +373,15 @@ sub findUniqueConstraints my $table = $row->{'table_name'}; my $columns = $row->{'constraint_definition'}; + # Test the opclass is BTree and was not added after installation + my @classes = split(/ /, $row->{'indclass'}); + while (my $class = pop(@classes)) + { + $opclass_sth->execute($class); + + next ITERATION if ($sth->rows == 0); + } + # Extract the columns from the index definition $columns =~ s|.*\(([^\)]+)\).*|$1|g; $columns =~ s|([^\s]+)[^\s]+_ops|$1|g; @@ -358,9 +422,9 @@ MSG sub findSerials { my $sql = qq{ - SELECT nspname - , relname - , attname + SELECT nspname AS nspname + , relname AS relname + , attname AS attname , adsrc FROM pg_catalog.pg_class as c