Re: WIP: a way forward on bootstrap data

From: John Naylor <jcnaylor(at)gmail(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: WIP: a way forward on bootstrap data
Date: 2017-12-14 10:59:12
Message-ID: CAJVSVGUGZhrPyRH4EyZmvid-PW04gODinyGSTcLt2NxHyy=PqQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 12/13/17, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> On 12/13/17 04:06, John Naylor wrote:
>> There doesn't seem to be any interest in bootstrap data at the moment,
>> but rather than give up just yet, I've added a couple features to make
>> a data migration more compelling:
>
> I took a brief look at your patches, and there appear to be a number of
> good cleanups in there at least. But could you please send patches in
> git format-patch format with commit messages, so we don't have to guess
> what each patch does?

Thanks for taking a look and for pointing me to git format-patch.
That's much nicer than trying to keep emails straight. I've attached a
new patchset.

Note that 4-7 and 9-10 are units as far as the build is concerned.
Meaning, once 4 is applied, the build is broken until 7 is applied.
Also, postgres.bki won't diff 100% clean with the master branch
because of some useless quotes in the latter.

One thing that occured to me while looking over patch 0004 again: It's
now a bit uglier to handle indexing.h and toasting.h. I think it might
be cleaner to keep those statements in the header of the catalog they
refer to. That has the additional benefit of making the headers the
Single Point of Truth for a catalog schema.

TODO:
-Docs and README
-Finish SQL generation patch
-Consider generating pg_type #defines

-John Naylor

Attachment Content-Type Size
0001-Fix-typos-and-oversights-in-catalog-headers_v3.patch text/x-patch 4.1 KB
0002-Cleanup-distprep-scripts_v3.patch text/x-patch 7.1 KB
0003-Remove-hard-coded-schema-knowledge-about-pg_attri_v3.patch text/x-patch 14.5 KB
0004-Data-conversion-infrastructure_v3.patch text/x-patch 53.9 KB
0005-Mechanical-data-conversion_v3.patch.tar.gz application/x-gzip 94.0 KB
0006-Hand-edits-of-data-files_v3.patch text/x-patch 69.6 KB
0007-Update-distprep-scripts_v3.patch text/x-patch 12.2 KB
0008-Remove-data-from-catalog-headers_v3.patch.tar.gz application/x-gzip 107.2 KB
0009-Replace-oids-with-human-readable-names_v3.patch.tar.gz application/x-gzip 122.7 KB
0010-Type-aliases-for-oid-lookups_v3.patch text/x-patch 12.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2017-12-14 11:08:10 Re: CUBE seems a bit confused about ORDER BY
Previous Message Masahiko Sawada 2017-12-14 10:45:48 Re: [HACKERS] Moving relation extension locks out of heavyweight lock manager