forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit52585f8
committed
Create common infrastructure for cross-version upgrade testing.
To test pg_upgrade across major PG versions, we have to be able tomodify or drop any old objects with no-longer-supported properties,and we have to be able to deal with cosmetic changes in pg_dump output.Up to now, the buildfarm and pg_upgrade's own test infrastructure hadseparate implementations of the former, and we had nothing but veryad-hoc rules for the latter (including an arbitrary threshold on howmany lines of unchecked diff were okay!). This patch creates a Perlmodule that can be shared by both those use-cases, and adds logicthat deals with pg_dump output diffs in a much more tightly definedfashion.This largely supersedes previous efforts in commits0df9641,9814ff5, and62be9e4, which developed a SQL-script-based solutionfor the task of dropping old objects. There was nothing fundamentallywrong with that work in itself, but it had no basis for solving theoutput-formatting problem. The most plausible way to deal withformatting is to build a Perl module that can perform editing on thedump files; and once we commit to that, it makes more sense for thesame module to also embed the knowledge of what has to be done fordropping old objects.Back-patch versions of the helper module as far as 9.2, tosupport buildfarm animals that still test that far back.It's also necessary to back-patch PostgreSQL/Version.pm,because the new code depends on that. I fixed up pg_upgrade's002_pg_upgrade.pl in v15, but did not look into back-patchingit further than that.Tom Lane and Andrew DunstanDiscussion:https://postgr.es/m/891521.1673657296@sss.pgh.pa.us1 parent980ae17 commit52585f8
File tree
4 files changed
+586
-190
lines changed- src
- bin/pg_upgrade
- t
- test/perl/PostgreSQL/Test
4 files changed
+586
-190
lines changedLines changed: 27 additions & 34 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
10 | 10 |
| |
11 | 11 |
| |
12 | 12 |
| |
13 |
| - | |
14 |
| - | |
15 |
| - | |
16 |
| - | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
17 | 18 |
| |
18 | 19 |
| |
19 |
| - | |
20 |
| - | |
21 |
| - | |
22 |
| - | |
23 |
| - | |
24 |
| - | |
25 |
| - | |
26 |
| - | |
27 |
| - | |
28 |
| - | |
29 |
| - | |
30 |
| - | |
31 |
| - | |
32 |
| - | |
33 |
| - | |
34 |
| - | |
35 |
| - | |
36 |
| - | |
37 |
| - | |
| 20 | + | |
38 | 21 |
| |
39 | 22 |
| |
40 | 23 |
| |
| |||
52 | 35 |
| |
53 | 36 |
| |
54 | 37 |
| |
55 |
| - | |
| 38 | + | |
56 | 39 |
| |
57 | 40 |
| |
58 | 41 |
| |
59 | 42 |
| |
60 | 43 |
| |
61 | 44 |
| |
62 |
| - | |
| 45 | + | |
63 | 46 |
| |
64 |
| - | |
65 |
| - | |
66 |
| - | |
67 |
| - | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
68 | 51 |
| |
69 |
| - | |
70 |
| - | |
71 |
| - | |
72 |
| - | |
73 |
| - | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + |
Lines changed: 35 additions & 41 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
10 | 10 |
| |
11 | 11 |
| |
12 | 12 |
| |
| 13 | + | |
13 | 14 |
| |
14 | 15 |
| |
15 | 16 |
| |
| |||
37 | 38 |
| |
38 | 39 |
| |
39 | 40 |
| |
40 |
| - | |
| 41 | + | |
41 | 42 |
| |
42 | 43 |
| |
43 |
| - | |
44 |
| - | |
45 |
| - | |
46 |
| - | |
47 |
| - | |
48 |
| - | |
49 |
| - | |
| 44 | + | |
50 | 45 |
| |
51 |
| - | |
52 |
| - | |
53 |
| - | |
54 |
| - | |
55 |
| - | |
56 |
| - | |
57 |
| - | |
58 |
| - | |
59 |
| - | |
60 |
| - | |
61 |
| - | |
62 |
| - | |
63 |
| - | |
64 |
| - | |
65 |
| - | |
66 |
| - | |
67 |
| - | |
68 |
| - | |
69 |
| - | |
70 |
| - | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
71 | 51 |
| |
72 | 52 |
| |
73 | 53 |
| |
| |||
83 | 63 |
| |
84 | 64 |
| |
85 | 65 |
| |
86 |
| - | |
| 66 | + | |
87 | 67 |
| |
88 | 68 |
| |
89 | 69 |
| |
| |||
198 | 178 |
| |
199 | 179 |
| |
200 | 180 |
| |
201 |
| - | |
202 |
| - | |
203 |
| - | |
204 |
| - | |
205 |
| - | |
206 |
| - | |
207 |
| - | |
208 |
| - | |
209 |
| - | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
210 | 204 |
| |
211 | 205 |
| |
212 | 206 |
| |
| |||
359 | 353 |
| |
360 | 354 |
| |
361 | 355 |
| |
362 |
| - | |
363 |
| - | |
| 356 | + | |
| 357 | + | |
364 | 358 |
| |
365 | 359 |
| |
366 | 360 |
| |
| |||
371 | 365 |
| |
372 | 366 |
| |
373 | 367 |
| |
374 |
| - | |
| 368 | + | |
375 | 369 |
| |
376 | 370 |
| |
377 | 371 |
| |
|
Lines changed: 0 additions & 115 deletions
This file was deleted.
0 commit comments
Comments
(0)