Fix regression.
[mediawiki.git] / maintenance / postgres / compare_schemas.pl
blob0a025bdcb45fe8c3b79bc2d00ad5d366e4ebbce0
1 #!/usr/bin/perl
3 ## Rough check that the base and postgres "tables.sql" are in sync
4 ## Should be run from maintenance/postgres
5 ## Checks a few other things as well...
7 use strict;
8 use warnings;
9 use Data::Dumper;
10 use Cwd;
12 #check_valid_sql();
14 my @old = ('../tables.sql');
15 my $new = 'tables.sql';
16 my @xfile;
18 ## Read in exceptions and other metadata
19 my %ok;
20 while (<DATA>) {
21 next unless /^(\w+)\s*:\s*([^#]+)/;
22 my ($name,$val) = ($1,$2);
23 chomp $val;
24 if ($name eq 'RENAME') {
25 die "Invalid rename\n" unless $val =~ /(\w+)\s+(\w+)/;
26 $ok{OLD}{$1} = $2;
27 $ok{NEW}{$2} = $1;
28 next;
30 if ($name eq 'XFILE') {
31 push @xfile, $val;
32 next;
34 for (split /\s+/ => $val) {
35 $ok{$name}{$_} = 0;
39 my $datatype = join '|' => qw(
40 bool
41 tinyint int bigint real float
42 tinytext mediumtext text char varchar varbinary binary
43 timestamp datetime
44 tinyblob mediumblob blob
46 $datatype .= q{|ENUM\([\"\w, ]+\)};
47 $datatype = qr{($datatype)};
49 my $typeval = qr{(\(\d+\))?};
51 my $typeval2 = qr{ signed| unsigned| binary| NOT NULL| NULL| PRIMARY KEY| AUTO_INCREMENT| default ['\-\d\w"]+| REFERENCES .+CASCADE};
53 my $indextype = join '|' => qw(INDEX KEY FULLTEXT), 'PRIMARY KEY', 'UNIQUE INDEX', 'UNIQUE KEY';
54 $indextype = qr{$indextype};
56 my $engine = qr{TYPE|ENGINE};
58 my $tabletype = qr{InnoDB|MyISAM|HEAP|HEAP MAX_ROWS=\d+|InnoDB MAX_ROWS=\d+ AVG_ROW_LENGTH=\d+};
60 my $charset = qr{utf8|binary};
62 open my $newfh, '<', $new or die qq{Could not open $new: $!\n};
65 my ($table,%old);
67 ## Read in the xfiles
68 my %xinfo;
69 for my $xfile (@xfile) {
70 print "Loading $xfile\n";
71 my $info = parse_sql($xfile);
72 for (keys %$info) {
73 $xinfo{$_} = $info->{$_};
77 for my $oldfile (@old) {
78 print "Loading $oldfile\n";
79 my $info = parse_sql($oldfile);
80 for (keys %xinfo) {
81 $info->{$_} = $xinfo{$_};
83 $old{$oldfile} = $info;
86 sub parse_sql {
88 my $oldfile = shift;
90 open my $oldfh, '<', $oldfile or die qq{Could not open $oldfile: $!\n};
92 my %info;
93 while (<$oldfh>) {
94 next if /^\s*\-\-/ or /^\s+$/;
95 s/\s*\-\- [\w ]+$//;
96 chomp;
98 if (/CREATE\s*TABLE/i) {
99 if (m{^CREATE TABLE /\*_\*/(\w+) \($}) {
100 $table = $1;
102 elsif (m{^CREATE TABLE /\*\$wgDBprefix\*/(\w+) \($}) {
103 $table = $1;
105 else {
106 die qq{Invalid CREATE TABLE at line $. of $oldfile\n};
108 $info{$table}{name}=$table;
110 elsif (m{^\) /\*\$wgDBTableOptions\*/}) {
111 $info{$table}{engine} = 'ENGINE';
112 $info{$table}{type} = 'variable';
114 elsif (/^\) ($engine)=($tabletype);$/) {
115 $info{$table}{engine}=$1;
116 $info{$table}{type}=$2;
118 elsif (/^\) ($engine)=($tabletype), DEFAULT CHARSET=($charset);$/) {
119 $info{$table}{engine}=$1;
120 $info{$table}{type}=$2;
121 $info{$table}{charset}=$3;
123 elsif (/^ (\w+) $datatype$typeval$typeval2{0,4},?$/) {
124 $info{$table}{column}{$1} = $2;
125 my $extra = $3 || '';
126 $info{$table}{columnfull}{$1} = "$2$extra";
128 elsif (m{^ UNIQUE KEY (\w+) \((.+?)\)}) {
130 elsif (m{^CREATE (?:UNIQUE )?(?:FULLTEXT )?INDEX /\*i\*/(\w+) ON /\*_\*/(\w+) \((.+?)\);}) {
132 elsif (m{^\s*PRIMARY KEY \([\w,]+\)}) {
134 else {
135 die "Cannot parse line $. of $oldfile:\n$_\n";
139 close $oldfh or die qq{Could not close "$oldfile": $!\n};
141 return \%info;
143 } ## end of parse_sql
145 ## Read in the parser test information
146 my $parsefile = '../parserTests.inc';
147 open my $pfh, '<', $parsefile or die qq{Could not open "$parsefile": $!\n};
148 my $stat = 0;
149 my %ptable;
150 while (<$pfh>) {
151 if (!$stat) {
152 if (/function listTables/) {
153 $stat = 1;
155 next;
157 $ptable{$1}=2 while m{'(\w+)'}g;
158 last if /\);/;
160 close $pfh or die qq{Could not close "$parsefile": $!\n};
162 my $OK_NOT_IN_PTABLE = '
163 change_tag
164 filearchive
165 logging
166 profiling
167 querycache_info
168 searchindex
169 tag_summary
170 trackbacks
171 transcache
172 user_newtalk
173 updatelog
174 valid_tag
177 ## Make sure all tables in main tables.sql are accounted for in the parsertest.
178 for my $table (sort keys %{$old{'../tables.sql'}}) {
179 $ptable{$table}++;
180 next if $ptable{$table} > 2;
181 next if $OK_NOT_IN_PTABLE =~ /\b$table\b/;
182 print qq{Table "$table" is in the schema, but not used inside of parserTest.inc\n};
184 ## Any that are used in ptables but no longer exist in the schema?
185 for my $table (sort grep { $ptable{$_} == 2 } keys %ptable) {
186 print qq{Table "$table" ($ptable{$table}) used in parserTest.inc, but not found in schema\n};
189 for my $oldfile (@old) {
191 ## Begin non-standard indent
193 ## MySQL sanity checks
194 for my $table (sort keys %{$old{$oldfile}}) {
195 my $t = $old{$oldfile}{$table};
196 if ($t->{engine} eq 'TYPE') {
197 die "Invalid engine for $oldfile: $t->{engine}\n" unless $t->{name} eq 'profiling';
199 my $charset = $t->{charset} || '';
200 if ($oldfile !~ /binary/ and $charset eq 'binary') {
201 die "Invalid charset for $oldfile: $charset\n";
205 my $dtype = join '|' => qw(
206 SMALLINT INTEGER BIGINT NUMERIC SERIAL
207 TEXT CHAR VARCHAR
208 BYTEA
209 TIMESTAMPTZ
210 CIDR
212 $dtype = qr{($dtype)};
213 my %new;
214 my ($infunction,$inview,$inrule,$lastcomma) = (0,0,0,0);
215 seek $newfh, 0, 0;
216 while (<$newfh>) {
217 next if /^\s*\-\-/ or /^\s*$/;
218 s/\s*\-\- [\w ']+$//;
219 next if /^BEGIN;/ or /^SET / or /^COMMIT;/;
220 next if /^CREATE SEQUENCE/;
221 next if /^CREATE(?: UNIQUE)? INDEX/;
222 next if /^CREATE FUNCTION/;
223 next if /^CREATE TRIGGER/ or /^ FOR EACH ROW/;
224 next if /^INSERT INTO/ or /^ VALUES \(/;
225 next if /^ALTER TABLE/;
226 chomp;
228 if (/^\$mw\$;?$/) {
229 $infunction = $infunction ? 0 : 1;
230 next;
232 next if $infunction;
234 next if /^CREATE VIEW/ and $inview = 1;
235 if ($inview) {
236 /;$/ and $inview = 0;
237 next;
240 next if /^CREATE RULE/ and $inrule = 1;
241 if ($inrule) {
242 /;$/ and $inrule = 0;
243 next;
246 if (/^CREATE TABLE "?(\w+)"? \($/) {
247 $table = $1;
248 $new{$table}{name}=$table;
249 $lastcomma = 1;
251 elsif (/^\);$/) {
252 if ($lastcomma) {
253 warn "Stray comma before line $.\n";
256 elsif (/^ (\w+) +$dtype.*?(,?)(?: --.*)?$/) {
257 $new{$table}{column}{$1} = $2;
258 if (!$lastcomma) {
259 print "Missing comma before line $. of $new\n";
261 $lastcomma = $3 ? 1 : 0;
263 elsif (m{^\s*PRIMARY KEY \([\w,]+\)}) {
264 $lastcomma = 0;
266 else {
267 die "Cannot parse line $. of $new:\n$_\n";
271 ## Which column types are okay to map from mysql to postgres?
272 my $COLMAP = q{
273 ## INTS:
274 tinyint SMALLINT
275 int INTEGER SERIAL
276 bigint BIGINT
277 real NUMERIC
278 float NUMERIC
280 ## TEXT:
281 varchar(15) TEXT
282 varchar(32) TEXT
283 varchar(70) TEXT
284 varchar(255) TEXT
285 varchar TEXT
286 text TEXT
287 tinytext TEXT
288 ENUM TEXT
290 ## TIMESTAMPS:
291 varbinary(14) TIMESTAMPTZ
292 binary(14) TIMESTAMPTZ
293 datetime TIMESTAMPTZ
294 timestamp TIMESTAMPTZ
296 ## BYTEA:
297 mediumblob BYTEA
299 ## OTHER:
300 bool SMALLINT # Sigh
303 ## Allow specific exceptions to the above
304 my $COLMAPOK = q{
305 ## User inputted text strings:
306 ar_comment tinyblob TEXT
307 fa_description tinyblob TEXT
308 img_description tinyblob TEXT
309 ipb_reason tinyblob TEXT
310 log_action varbinary(10) TEXT
311 oi_description tinyblob TEXT
312 rev_comment tinyblob TEXT
313 rc_log_action varbinary(255) TEXT
314 rc_log_type varbinary(255) TEXT
316 ## Simple text-only strings:
317 ar_flags tinyblob TEXT
318 ct_params blob TEXT
319 fa_minor_mime varbinary(32) TEXT
320 fa_storage_group varbinary(16) TEXT # Just 'deleted' for now, should stay plain text
321 fa_storage_key varbinary(64) TEXT # sha1 plus text extension
322 ipb_address tinyblob TEXT # IP address or username
323 ipb_range_end tinyblob TEXT # hexadecimal
324 ipb_range_start tinyblob TEXT # hexadecimal
325 img_minor_mime varbinary(32) TEXT
326 img_sha1 varbinary(32) TEXT
327 job_cmd varbinary(60) TEXT # Should we limit to 60 as well?
328 keyname varbinary(255) TEXT # No tablename prefix (objectcache)
329 ll_lang varbinary(20) TEXT # Language code
330 log_params blob TEXT # LF separated list of args
331 log_type varbinary(10) TEXT
332 ls_field varbinary(32) TEXT
333 oi_minor_mime varbinary(32) TEXT
334 oi_sha1 varbinary(32) TEXT
335 old_flags tinyblob TEXT
336 old_text mediumblob TEXT
337 pp_propname varbinary(60) TEXT
338 pp_value blob TEXT
339 page_restrictions tinyblob TEXT # CSV string
340 pf_server varchar(30) TEXT
341 pr_level varbinary(60) TEXT
342 pr_type varbinary(60) TEXT
343 pt_create_perm varbinary(60) TEXT
344 pt_reason tinyblob TEXT
345 qc_type varbinary(32) TEXT
346 qcc_type varbinary(32) TEXT
347 qci_type varbinary(32) TEXT
348 rc_params blob TEXT
349 rlc_to_blob blob TEXT
350 ts_tags blob TEXT
351 ug_group varbinary(16) TEXT
352 up_property varbinary(32) TEXT
353 up_value blob TEXT
354 user_email_token binary(32) TEXT
355 user_ip varbinary(40) TEXT
356 user_newpassword tinyblob TEXT
357 user_options blob TEXT
358 user_password tinyblob TEXT
359 user_token binary(32) TEXT
361 ## Text URLs:
362 el_index blob TEXT
363 el_to blob TEXT
364 iw_url blob TEXT
365 tb_url blob TEXT
366 tc_url varbinary(255) TEXT
368 ## Deprecated or not yet used:
369 ar_text mediumblob TEXT
370 job_params blob TEXT
371 log_deleted tinyint INTEGER # Not used yet, but keep it INTEGER for safety
372 rc_type tinyint CHAR
374 ## Number tweaking:
375 fa_bits int SMALLINT # bits per pixel
376 fa_height int SMALLINT
377 fa_width int SMALLINT # Hope we don't see an image this wide...
378 hc_id int BIGINT # Odd that site_stats is all bigint...
379 img_bits int SMALLINT # bits per image should stay sane
380 oi_bits int SMALLINT
382 ## True binary fields, usually due to gzdeflate and/or serialize:
383 math_inputhash varbinary(16) BYTEA
384 math_outputhash varbinary(16) BYTEA
386 ## Namespaces: not need for such a high range
387 ar_namespace int SMALLINT
388 job_namespace int SMALLINT
389 log_namespace int SMALLINT
390 page_namespace int SMALLINT
391 pl_namespace int SMALLINT
392 pt_namespace int SMALLINT
393 qc_namespace int SMALLINT
394 rc_namespace int SMALLINT
395 rd_namespace int SMALLINT
396 rlc_to_namespace int SMALLINT
397 tl_namespace int SMALLINT
398 wl_namespace int SMALLINT
400 ## Easy enough to change if a wiki ever does grow this big:
401 ss_active_users bigint INTEGER
402 ss_good_articles bigint INTEGER
403 ss_total_edits bigint INTEGER
404 ss_total_pages bigint INTEGER
405 ss_total_views bigint INTEGER
406 ss_users bigint INTEGER
408 ## True IP - keep an eye on these, coders tend to make textual assumptions
409 rc_ip varbinary(40) CIDR # Want to keep an eye on this
411 ## Others:
412 tc_time int TIMESTAMPTZ
417 my %colmap;
418 for (split /\n/ => $COLMAP) {
419 next unless /^\w/;
420 s/(.*?)#.*/$1/;
421 my ($col,@maps) = split / +/, $_;
422 for (@maps) {
423 $colmap{$col}{$_} = 1;
427 my %colmapok;
428 for (split /\n/ => $COLMAPOK) {
429 next unless /^\w/;
430 my ($col,$old,$new) = split / +/, $_;
431 $colmapok{$col}{$old}{$new} = 1;
434 ## Old but not new
435 for my $t (sort keys %{$old{$oldfile}}) {
436 if (!exists $new{$t} and !exists $ok{OLD}{$t}) {
437 print "Table not in $new: $t\n";
438 next;
440 next if exists $ok{OLD}{$t} and !$ok{OLD}{$t};
441 my $newt = exists $ok{OLD}{$t} ? $ok{OLD}{$t} : $t;
442 my $oldcol = $old{$oldfile}{$t}{column};
443 my $oldcolfull = $old{$oldfile}{$t}{columnfull};
444 my $newcol = $new{$newt}{column};
445 for my $c (keys %$oldcol) {
446 if (!exists $newcol->{$c}) {
447 print "Column $t.$c not in $new\n";
448 next;
451 for my $c (sort keys %$newcol) {
452 if (!exists $oldcol->{$c}) {
453 print "Column $t.$c not in $oldfile\n";
454 next;
456 ## Column types (roughly) match up?
457 my $new = $newcol->{$c};
458 my $old = $oldcolfull->{$c};
460 ## Known exceptions:
461 next if exists $colmapok{$c}{$old}{$new};
463 $old =~ s/ENUM.*/ENUM/;
464 if (! exists $colmap{$old}{$new}) {
465 print "Column types for $t.$c do not match: $old does not map to $new\n";
469 ## New but not old:
470 for (sort keys %new) {
471 if (!exists $old{$oldfile}{$_} and !exists $ok{NEW}{$_}) {
472 print "Not in $oldfile: $_\n";
473 next;
478 } ## end each file to be parsed
481 sub check_valid_sql {
483 ## Check for a few common problems in most php files
485 my $olddir = getcwd();
486 chdir("../..");
487 for my $basedir (qw/includes extensions/) {
488 scan_dir($basedir);
490 chdir $olddir;
492 return;
494 } ## end of check_valid_sql
497 sub scan_dir {
499 my $dir = shift;
501 opendir my $dh, $dir or die qq{Could not opendir $dir: $!\n};
502 #print "Scanning $dir...\n";
503 for my $file (grep { -f "$dir/$_" and /\.php$/ } readdir $dh) {
504 find_problems("$dir/$file");
506 rewinddir $dh;
507 for my $subdir (grep { -d "$dir/$_" and ! /\./ } readdir $dh) {
508 scan_dir("$dir/$subdir");
510 closedir $dh or die qq{Closedir failed: $!\n};
511 return;
513 } ## end of scan_dir
515 sub find_problems {
517 my $file = shift;
518 open my $fh, '<', $file or die qq{Could not open "$file": $!\n};
519 my $lastline = '';
520 my $inarray = 0;
521 while (<$fh>) {
522 if (/FORCE INDEX/ and $file !~ /Database\w*\.php/) {
523 warn "Found FORCE INDEX string at line $. of $file\n";
525 if (/REPLACE INTO/ and $file !~ /Database\w*\.php/) {
526 warn "Found REPLACE INTO string at line $. of $file\n";
528 if (/\bIF\s*\(/ and $file !~ /DatabaseMySQL\.php/) {
529 warn "Found IF string at line $. of $file\n";
531 if (/\bCONCAT\b/ and $file !~ /Database\w*\.php/) {
532 warn "Found CONCAT string at line $. of $file\n";
534 if (/\bGROUP\s+BY\s*\d\b/i and $file !~ /Database\w*\.php/) {
535 warn "Found GROUP BY # at line $. of $file\n";
537 if (/wfGetDB\s*\(\s+\)/io) {
538 warn "wfGETDB is missing parameters at line $. of $file\n";
540 if (/=\s*array\s*\(\s*$/) {
541 $inarray = 1;
542 next;
544 if ($inarray) {
545 if (/\s*\);\s*$/) {
546 $inarray = 0;
547 next;
549 next if ! /\w/ or /array\(\s*$/ or /^\s*#/ or m{^\s*//};
550 if (! /,/) {
551 my $nextline = <$fh>;
552 last if ! defined $nextline;
553 if ($nextline =~ /^\s*\)[;,]/) {
554 $inarray = 0;
555 next;
557 #warn "Array is missing a comma? Line $. of $file\n";
561 close $fh or die qq{Could not close "$file": $!\n};
562 return;
564 } ## end of find_problems
567 __DATA__
568 ## Known exceptions
569 OLD: searchindex ## We use tsearch2 directly on the page table instead
570 RENAME: user mwuser ## Reserved word causing lots of problems
571 RENAME: text pagecontent ## Reserved word
572 NEW: mediawiki_version ## Just us, for now
573 XFILE: ../archives/patch-profiling.sql