From: | Noah Misch <noah(at)leadboat(dot)com> |
---|---|
To: | Amir Rohan <amir(dot)rohan(at)mail(dot)com> |
Cc: | pgsql-hacker mailing list <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Patch: Revised documentation on base backups |
Date: | 2016-01-02 03:59:36 |
Message-ID: | 20160102035936.GA2957111@tornado.leadboat.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Sep 28, 2015 at 07:17:39AM +0300, Amir Rohan wrote:
> The docs seem to be in need of some improvement, primarily because
> it has the feel of having been monkey-patched repeatedly it in the
> past. It needs to have someone with more experience read through it
> and comment on technical accuracy and any knee-slapper error I
> might have introduced.
That's mostly outside my area, but I can answer one of your questions:
> One pain point in rewriting is the definition of terms, or lack
> there of in the manual. I didn't find a section defining terms
> conclusively for consistent referencing throughout the documentation.
> For example, how should one refer to the directory housing
> "postgresql.conf"?, is it:
>
> 1. The data directory
> 2. The storage directory
> 3. The PGDATA directory
> 4. The cluster directory
> 5. The server root directory
> 6. The config file directory
> 7. etc'
We have no standard term for "the directory housing 'postgresql.conf'". Since
the config_file setting customizes that directory independent of all other
directories of interest, it is not a reusable concept. Of the names you list,
"data directory" is the dominant term. (The data directory does not
necessarily hold configuration files, though it does by default.) "cluster
directory" appears a few times; those could just as easily say "data
directory". The other terms see little or no use.
> + To aid you in doing this, the base backup process creates a
> + a textual file called <filename>backup_label</> when you initiate the
> + base backup, it in the same directory as your server's
> + configuration files.
> + This places the database in backup mode, creating the
> + <filename>backup_label</>
> + file described earlier, as well as the <firstterm>tablespace map</>
> + file called
> + <filename>tablespace_map</> which contains information about tablespace
> + symbolic links in <filename>pg_tblspc/</>, if such links are present.
> + Both files are created in your cluster directory, alongside
> + your configuration files, and are critical to the integrity of
> + the backup.
I would write "<...> created in the data directory and are critical to <...>".
backup_label and tablespace_map go there (setting "data_directory") regardless
of configuration file location.
Thanks,
nm
From | Date | Subject | |
---|---|---|---|
Next Message | Shay Rojansky | 2016-01-02 07:06:01 | Re: Some 9.5beta2 backend processes not terminating properly? |
Previous Message | Noah Misch | 2016-01-02 03:28:10 | Re: pg_dump LOCK TABLE ONLY question |