| From: | Noah Misch <noah(at)leadboat(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Andres Freund <andres(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Jeff Davis <pgsql(at)j-davis(dot)com>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: mvcc catalo gsnapshots and TopTransactionContext |
| Date: | 2013-08-09 21:14:51 |
| Message-ID: | 20130809211451.GA469413@tornado.leadboat.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Aug 09, 2013 at 02:11:46PM -0400, Tom Lane wrote:
> Cache invalidation during abort should *not* lead to any attempt to
> immediately revalidate the cache. No amount of excuses will make that
> okay. I have not looked to see just what the path of control is in this
> particular case, but we need to fix it, not paper over it.
+1. What if (sub)transaction end only manipulated the local invalidation
message queue for later processing? Actual processing would happen after
CleanupSubTransaction() returns control to the owning xact, or at the start of
the next transaction for a top-level ending.
--
Noah Misch
EnterpriseDB http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Noah Misch | 2013-08-09 22:40:08 | Re: Proposal: leave a hint when switching logging away from stderr |
| Previous Message | Stefan Kaltenbrunner | 2013-08-09 20:37:21 | Re: [PATCH] Statistics collection for CLUSTER command |