svk-commitdCGXq.tmp

Add views to mysql producer, thanks groditi

Applied patches written by Nigel Metheringham.  His notes follow.
---

The postgres producer uses a hash as both a single level and a 2 level
symbol table for ensuring names are unique which causes some interesting
failures when a database has constraint names which match table names.

I've replaced the code with effectively the same code as the mysql
producer uses, massaged very slightly so as in the most cases to
produce the same name remapping as the old code but without the
extra explosions when it attempts to deref a scalar as a hash.

---

When a table has more than one index, all but the last CREATE INDEX
commands produced have a doubled semi-colon at the end of them.

This causes the parser for postgresql to explode :-)

One line bug fix appears to sort it.

Added patch from groditi adding SET type support to the mysql producer

Patch from ribasushi: Correctly graph self-referential constraints

Make Schema::Graph only load if "as_graph" is called on a Schema object

Applied patch from Ryan to uniqify index names sanely for the mysql producer

Made change suggested by Daniel Böhringer to allow "ID" integer DEFAULT nextval(('"AlleStudien_ID_seq"'::text)::regclass), to parse properly

Support uppercase foreign key target-columns.  Sent in by Daniel Boehringer

Implemented add_field, only the field is added, nothing more for now.

Adding tests for Oracle->add_field

Small fix in delay_constraints (missing ;).

Fix ORA-01442: column to be modified to NOT NULL is already NOT NULL

Including new test and test data into MANIFEST

alter_field implemented. alter_field test Pass.

Test for oracle alter_field

Declares the new tests in the MANIFEST, remove useless warnings

Define a name for pk constraint when delay_constraints is on

Document delay_constraints producer_args option

producer_args->{delay_constraints} can be used to add primary keys later

Fix weird bug, caused by a double evaluation in ternary if

implements options in oracle indexes

waiting for patch to be applied

Fix suprious diff on cols with charsets/collates (MySQL)

Add support for proper boolean fields in the mysql producer, as of v4.x

Fix false-diffing due to order of table options

Updated change file

Added support for proper enums under pg (as of 8.3), with pg version check, and deferrable constraints
Patch from Debolaz

Allow quote and other option to be passed to the producers.

0.09000 not 0.0900 (cpan gets confused if numbers go downwards)

Remove breakpoint

Update changes with latest release date

Add DBI dep (oops)
Update versions for 0.0900

Fix drop indexes for uniq constraints

Allow DEFAULT CHARACTER SET without '=' (as produced by mysqldump)

Add support for COLLATE table option to MySQL parser

Update changes/version for 0.0899_02

Oops, fix tests for fixed syntax

Fix syntax for index dropping in sqlite producer

Document significance of preproces_schema method

Fix warning messages

Update oracle producer, patch from plu.
Update postgres producer, patch from wreis

Merging back Diff refactor

Branch for diff refactoring
r11566@proteus (orig r1319):  ashberlin | 2007-11-27 13:34:56 +0000
MAss diff changes imported from Ash's local diff-refactor branch
r12366@proteus (orig r1320):  ashberlin | 2007-12-13 17:53:49 +0000
Some work on sanely normalizing fields
r12369@proteus (orig r1321):  ashberlin | 2007-12-16 22:24:00 +0000
Some SQL_TYPE mapping stuff
r12748@proteus (orig r1324):  ashberlin | 2007-12-20 16:54:56 +0000
Fix some more normalization problems
r12749@proteus (orig r1325):  ashberlin | 2007-12-20 17:23:44 +0000
Remove breakpoints
r12798@proteus (orig r1327):  ashberlin | 2007-12-21 10:02:30 +0000
Fix test
r13120@proteus (orig r1328):  ashberlin | 2008-01-10 13:55:19 +0000
Correct constraint names in preprocess for MySQL producer
r13152@proteus (orig r1329):  ashberlin | 2008-01-10 15:12:57 +0000
Move more normalization changes to preprocess_schema
r13244@proteus (orig r1331):  ashberlin | 2008-01-12 13:47:18 +0000
Fix after typo in merge
r13297@proteus (orig r1332):  ashberlin | 2008-01-15 23:34:54 +0000
Add renamed_from to tables.
r13309@proteus (orig r1333):  ashberlin | 2008-01-16 22:44:29 +0000
Start transactions in a portable manner
r13310@proteus (orig r1334):  ashberlin | 2008-01-16 23:47:59 +0000
Work round MySQL/InnoDB bug http://bugs.mysql.com/bug.php?id=13741
r13311@proteus (orig r1335):  ashberlin | 2008-01-16 23:50:00 +0000
Fix BEGIN in sqlite diff test
r13677@proteus (orig r1336):  ashberlin | 2008-01-21 16:22:51 +0000
Better tests (and fix bug) in batch alter with renamed tables + constraints

Better tests (and fix bug) in batch alter with renamed tables + constraints

Fix BEGIN in sqlite diff test

Work round MySQL/InnoDB bug bugs.mysql.com/bug.php?id=13741

Start transactions in a portable manner

Add renamed_from to tables.

Fix after typo in merge

Added an 'alter sequence' line to the parser grammer which will simply skip over an alter sequence statement.
Added a select section to the parser.  This is to handle function calls that
may affect the tables during creation.

Added the setval function.  Unfortunately, nothing is being done with this
because the parser doesn't seem to handle sequences.

Skipping all COPY commands because they are not needed for schema creation.

Added a line each for revoking and granting permissions for a schema.  This
doesn't actually do anything with that information.

Applied patch submitted by Nathan Gray

Removes periods in an index/constraint name caused by schema.tablename being appended to the name.

Strips a schema from the table name during index/constraint creation because a schema-qualified table name is not allowed.
r12687@metis (orig r1322):  mwz444 | 2007-12-18 16:20:28 +0000
r12747@metis (orig r1323):  mwz444 | 2007-12-20 15:48:09 +0000
r12797@metis (orig r1326):  mwz444 | 2007-12-20 20:07:22 +0000

Move more normalization changes to preprocess_schema

Correct constraint names in preprocess for MySQL producer

Fix test

Applied patch submitted by Nathan Gray

Removes periods in an index/constraint name caused by schema.tablename being appended to the name.

Strips a schema from the table name during index/constraint creation because a schema-qualified table name is not allowed.

Remove breakpoints

Fix some more normalization problems

Added a select section to the parser.  This is to handle function calls that
may affect the tables during creation.

Added the setval function.  Unfortunately, nothing is being done with this
because the parser doesn't seem to handle sequences.

Skipping all COPY commands because they are not needed for schema creation.

Added a line each for revoking and granting permissions for a schema.  This
doesn't actually do anything with that information.

Added an 'alter sequence' line to the parser grammer which will simply skip over an alter sequence statement.

Some SQL_TYPE mapping stuff

Some work on sanely normalizing fields

MAss diff changes imported from Ash's local diff-refactor branch

Branch for diff refactoring

Added fixes submitted by Peter Rabbitson:

Fix POD incorrectly referring to argument show_sizes as show_col_sizes
Fix POD formatting errors (=item instead of ==item)
Fix and improve handling of the skip_fields argument (silence warning)

Allows schema-qualified table names.
Fix suggested by Ken Clark.

Changed the name of unnamed foreign key constraints to TABLENAME_fk.

Fixed a problem where the deferrable definition required to have a "not".

Only run test if Graph::Directed installed

0.0899_01 diffing fixes

Ignore all TT test while TT is broken

Fix to not test while TT is broken

Committing patches sent by Florian Helmberger.  Here are his remarks:

Here are two patches for a few bugs in the PostgreSQL parser and producer.

The patch for lib/SQL/Translator/Parser/DBI/PostgreSQL.pm fixes a simple
typo which resulted in the complete absence of the field types and
corrects the way is_nullable is set so the producer now correctly flags
NOT NULL fields.

The patch for lib/SQL/Translator/Producer/PostgreSQL.pm adds missing
newlines to the last line of the 'Table:' and 'Comments:' headers. Also
it fixes an 'use of uninizialised value' warning in 359.

Before the patch the output for command

sqlt --from DBI --dsn dbi:Pg:dbname=testdb --to PostgreSQL \
--db-user postgres

looked like:

====
--
-- Table: mailaliases
--CREATE TABLE "mailaliases" (
"aliasname" (204),
"aliasto" (204),
"domain" (132),
"id" ,
"isdefault" ,
"datecreate" ,
Constraint "foo" UNIQUE ("isdefault")
);
====

After the patch it looks like:

====
--
-- Table: mailaliases
--
CREATE TABLE "mailaliases" (
"aliasname" character varying(204),
"aliasto" character varying(204),
"domain" character varying(132),
"id" int4 NOT NULL,
"isdefault" int4,
"datecreate" text,
Constraint "foo" UNIQUE ("isdefault")
);
====

Applied Hilmar's patches.

Ignore views and procedures with no text (weeds out exatraneous results from SQL Server 2005)

Added command-line option for MySQL parser version arg

Added options for ignoring the differences in SQL for views and procedures

Added cursory parsing of views and procedures

Added mysql-parser-version command-line option

Added cursory parsing of view, functions, and procedures (oh why!)
Added pre-processing for /*! comments from mysqldump
Added mysql_parser_version parser arg for enabling parsing not-before-version comments from mysqldump

Removed unused rules procedure_body and view_body

Small change to ordering of view and procedure diffs

Added diff comparison of views and procedures

Hacks for comparing sql attribute in equals() to ignore unimportant differences

Removed mistakenly checked in debug code

Added cursory parsing of procedures, functions, and views (oh my!)

Added GO to output after views and procedures

Helps to actually pass the output-db option

Added output_db option
Removed Oracle SYS_ constraints hacks

Made fields check in equals() ignore ordering

Added output-db option

Corrected index parsing on Oracle so every index is not a unique constraint

Skip constraint and index comparison shortcuts when ignoring the relevant names

Don't test is_valid in equals()

Added options to sqlt-diff to ignore index and/or constraint name differences

Changed Constraint.equals() to ignore ordering of fields and reference fields

Tweak previous comment parsing tweak for multiple lines

Slight change to comment parsing to allow asterisks in comments

Added linefeeds so beginning of create view/procedure statements don't end up commented out

Fix bug in YAML test

Patched mysql producer to name constraints sanely

Update changes, oops