Re: Skytools committed without hackers discussion/review

From: Hannu Krosing <hannu(at)skype(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Marko Kreen <markokr(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, Jan Wieck <JanWieck(at)yahoo(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Skytools committed without hackers discussion/review
Date: 2007-10-10 22:48:14
Message-ID: 1192056494.8959.60.camel@hannu-laptop
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Ühel kenal päeval, K, 2007-10-10 kell 17:17, kirjutas Tom Lane:
> "Marko Kreen" <markokr(at)gmail(dot)com> writes:
> > Could you describe bit more? The is_visible_txid() works
> > on data returned by txid_current_snapshot()? How can there
> > be any subtrans id's if txid_current_snapshot() wont return
> > them?
>
> Ah, I see: txid_current() never reports a subxact ID so there's no need to
> consider them elsewhere in txids either. OK, but this desperately needs
> to be documented.
>
> BTW, I notice that use of txid_current will force assignment of an XID
> in a transaction that might not otherwise have one. Does this matter,
> or is the expectation that it's only going to be used in transactions
> that are making DB modifications anyway?

yes, the common use in Skytools (and now in Slony) is in
insert/update/delete triggers to add current txid to log records.

> regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: You can help support the PostgreSQL project by donating at
>
> http://www.postgresql.org/about/donate

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Joshua D. Drake 2007-10-10 23:22:38 Re: Skytools committed without hackers discussion/review
Previous Message Hannu Krosing 2007-10-10 22:43:16 Re: Skytools committed without hackers discussion/review

Browse pgsql-hackers by date

  From Date Subject
Next Message Joshua D. Drake 2007-10-10 23:22:38 Re: Skytools committed without hackers discussion/review
Previous Message Hannu Krosing 2007-10-10 22:43:16 Re: Skytools committed without hackers discussion/review