On 30.10.20 17:45, Erik Rijkers wrote:
> Hi,
>
> I applied 0009-architecture-vs-master.patch to head
> and went through architecture.sgml (only that file),
> then produced the attached .diff
>
>
> And I wrote down some separate items:
>
> 1.
> 'Two Phase Locking' and 'TPL' should be, I think,
> 'Two-Phase Commit'. Please someone confirm.
> (no changes made)
>
> 2.
> To compare xid to sequence because they similarly 'count up' seems a
> bad idea.
> (I don't think it's always true in the case of sequences)
> (no changes made)
>
> 3.
> 'accesses' seems a somewhat strange word most of the time just
> 'access' may be better. Not sure - native speaker wanted. (no changes
> made)
>
> 4.
> 'heap', in postgres, means often (always?) files. But more generally,
> the meaning is more associated with memory. Therefore it would be
> good I think to explicitly use 'heap file' at least in the beginning
> once to make clear that heap implies 'safely written away to disk'.
> Again, I'm not quite sure if my understanding is correct - I have made
> no changes in this regard.
>
>
>
> Erik Rijkers
All suggestions so far are summarized in the attached patch with the
following exceptions:
- 'Two Phase Locking' is the intended term.
- Not adopted:
Second, the transfer of dirty buffers from Shared Memory to
files must take place. This is the primary task of the
- Background Writer process. Because I/O activities can block
+ Checkpointer process. Because I/O activities can block
other processes, it starts periodically and
Partly adopted:
- the data in the old version of the row does not change! ...
- before. Nothing is thrown away so far! Only <literal>xmax</literal> ...
--
J. Purtz
