Re: Replication identifiers, take 4

On 04/17/2015 11:54 AM, Andres Freund wrote:
> I've attached a rebased patch, that adds decision about origin logging
> to the relevant XLogInsert() callsites for "external" 2 byte identifiers
> and removes the pad-reusing version in the interest of moving forward.

Putting aside the 2 vs. 4 byte identifier issue, let's discuss naming:

I just realized that it talks about "replication identifier" as the new 
fundamental concept. The system table is called 
"pg_replication_identifier". But that's like talking about "index 
identifiers", instead of just indexes, and calling the system table 
pg_index_oid.

The important concept this patch actually adds is the *origin* of each 
transaction. That term is already used in some parts of the patch. I 
think we should roughly do a search-replace of "replication identifier" 
-> "replication origin" to the patch. Or even "transaction origin".

- Heikki