On Sun, 2012-03-04 at 16:39 +, Simon Riggs wrote:
> >> v3 attached.
Added to next commitfest.
Regards,
Jeff Davis
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
On Fri, Mar 9, 2012 at 2:59 PM, Simon Riggs wrote:
> On Fri, Mar 9, 2012 at 3:46 AM, Robert Haas wrote:
>> On Wed, Mar 7, 2012 at 5:41 PM, Simon Riggs wrote:
Case #2 is certainly a problem for FrozenXID as well, because anything
that's marked with FrozenXID is going to look visible to
On Fri, Mar 9, 2012 at 3:46 AM, Robert Haas wrote:
> On Wed, Mar 7, 2012 at 5:41 PM, Simon Riggs wrote:
>>> Case #2 is certainly a problem for FrozenXID as well, because anything
>>> that's marked with FrozenXID is going to look visible to everybody,
>>> including our older snapshots. And I gath
On Wed, Mar 7, 2012 at 5:41 PM, Simon Riggs wrote:
>> Case #2 is certainly a problem for FrozenXID as well, because anything
>> that's marked with FrozenXID is going to look visible to everybody,
>> including our older snapshots. And I gather you're saying it's also a
>> problem for HEAP_XMIN_COM
On Wed, Mar 7, 2012 at 8:21 PM, Robert Haas wrote:
> On Wed, Mar 7, 2012 at 2:06 PM, Simon Riggs wrote:
>>> I am not thrilled with the design as it stands, but bulk loading is a
>>> known and serious pain point for us, so it would be awfully nice to
>>> improve it. I'm not sure whether we should
On Wed, Mar 7, 2012 at 2:06 PM, Simon Riggs wrote:
>> I am not thrilled with the design as it stands, but bulk loading is a
>> known and serious pain point for us, so it would be awfully nice to
>> improve it. I'm not sure whether we should only go as far as setting
>> HEAP_XMIN_COMMITTED or whet
On Wed, Mar 7, 2012 at 5:39 PM, Robert Haas wrote:
> On Wed, Mar 7, 2012 at 10:26 AM, Simon Riggs wrote:
>> On Wed, Mar 7, 2012 at 2:59 PM, Robert Haas wrote:
>>> All true.
>>
>> So gentlemen, do we think this is worth pursuing further for this release?
>>
>> I'm sure usual arguments apply all r
On Wed, Mar 7, 2012 at 10:26 AM, Simon Riggs wrote:
> On Wed, Mar 7, 2012 at 2:59 PM, Robert Haas wrote:
>> All true.
>
> So gentlemen, do we think this is worth pursuing further for this release?
>
> I'm sure usual arguments apply all round, so I'll skip that part.
This patch is awfully late to
On Wed, Mar 7, 2012 at 2:59 PM, Robert Haas wrote:
> All true.
So gentlemen, do we think this is worth pursuing further for this release?
I'm sure usual arguments apply all round, so I'll skip that part.
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24
On Wed, Mar 7, 2012 at 7:49 AM, Noah Misch wrote:
> On Tue, Mar 06, 2012 at 08:36:05AM -0500, Robert Haas wrote:
>> On Tue, Mar 6, 2012 at 5:43 AM, Noah Misch wrote:
>> >> Well, consider something like CLUSTER. ?It's perfectly OK for CLUSTER
>> >> to operate on a table that has been truncated sin
On Tue, Mar 06, 2012 at 08:36:05AM -0500, Robert Haas wrote:
> On Tue, Mar 6, 2012 at 5:43 AM, Noah Misch wrote:
> >> Well, consider something like CLUSTER. ?It's perfectly OK for CLUSTER
> >> to operate on a table that has been truncated since CLUSTER's snapshot
> >> was taken, and no serializati
On Tue, Mar 6, 2012 at 5:43 AM, Noah Misch wrote:
>> Now, maybe we're never going to fix those kinds of anomalies anyway,
>> but if we go with this architecture, then I think the chances of it
>> ever being palatable to try are pretty low.
>
> Why?
Because it'll require at least one XID column in
On Mon, Mar 05, 2012 at 03:46:16PM -0500, Robert Haas wrote:
> On Mon, Mar 5, 2012 at 2:22 PM, Noah Misch wrote:
> > I can see this strategy applying to many relation-pertinent system catalogs.
> > Do you foresee applications to non-relation catalogs?
>
> Well, in theory, we have similar issues i
On Sun, Mar 04, 2012 at 01:02:57PM +, Simon Riggs wrote:
> More detailed thoughts show that the test in heap_beginscan_internal()
> is not right enough, i.e. wrong.
>
> We need a specific XidInMVCCSnapshot test on the relvalidxid, so it
> needs to be a specific xid, not an xmin because otherwi
On Mon, Mar 5, 2012 at 8:46 PM, Robert Haas wrote:
>> In any event, I think a pg_class.relvalidxmin is the right starting point.
>> One might imagine a family of relvalidxmin, convalidxmin, indcheckxmin
>> (already exists), inhvalidxmin, and attvalidxmin. relvalidxmin is like the
>> AccessExclus
On Mon, Mar 5, 2012 at 2:22 PM, Noah Misch wrote:
> On Mon, Feb 13, 2012 at 09:29:56AM -0500, Robert Haas wrote:
>> On Fri, Feb 10, 2012 at 11:46 PM, Noah Misch wrote:
>> > I've yet to see an MVCC anomaly that one can reproduce at REPEATABLE READ
>> > and
>> > not at READ COMMITTED. ?They tend t
On Mon, Mar 5, 2012 at 5:49 PM, Robert Haas wrote:
> Well, there's no point that I can see in having two checks. I just
> dislike the idea that we have to remember to add this check for every
> method of accessing the relation - doesn't seem terribly future-proof.
> It gets even worse if you st
On Mon, Feb 13, 2012 at 09:29:56AM -0500, Robert Haas wrote:
> On Fri, Feb 10, 2012 at 11:46 PM, Noah Misch wrote:
> > I've yet to see an MVCC anomaly that one can reproduce at REPEATABLE READ
> > and
> > not at READ COMMITTED. ?They tend to be narrow race conditions at READ
> > COMMITTED, yet ea
On Mon, Mar 5, 2012 at 12:42 PM, Simon Riggs wrote:
> On Mon, Mar 5, 2012 at 4:46 PM, Simon Riggs wrote:
>>> It does not seem right that the logic for detecting the serialization
>>> error is in heap_beginscan_internal(). Surely this is just as much of
>>> a problem for an index-scan or index-on
On Mon, Mar 5, 2012 at 11:46 AM, Simon Riggs wrote:
> I agree behaviour is wrong, the only question is whether our users
> rely in some way on that behaviour. Given the long discussion on that
> point earlier I thought it best to add a GUC. Easy to remove, now or
> later.
AFAICT, all the discussi
On Mon, Mar 5, 2012 at 4:46 PM, Simon Riggs wrote:
>> It does not seem right that the logic for detecting the serialization
>> error is in heap_beginscan_internal(). Surely this is just as much of
>> a problem for an index-scan or index-only-scan.
>
> err, very good point. Doh.
>
>> We don't wan
On Mon, Mar 5, 2012 at 4:32 PM, Robert Haas wrote:
> On Sun, Mar 4, 2012 at 11:39 AM, Simon Riggs wrote:
>> Marti, please review this latest version which has new isolation tests added.
>>
>> This does both TRUNCATE and CREATE TABLE.
>
> I don't see any need for a GUC to control this behavior. T
On Sun, Mar 4, 2012 at 11:39 AM, Simon Riggs wrote:
> Marti, please review this latest version which has new isolation tests added.
>
> This does both TRUNCATE and CREATE TABLE.
I don't see any need for a GUC to control this behavior. The current
behavior is wrong, so if we're going to choose th
On Sun, Mar 4, 2012 at 1:02 PM, Simon Riggs wrote:
> On Sun, Mar 4, 2012 at 9:59 AM, Simon Riggs wrote:
>> On Sun, Mar 4, 2012 at 2:28 AM, Marti Raudsepp wrote:
>>> On Sat, Mar 3, 2012 at 14:53, Simon Riggs wrote:
Thanks Noah for drawing attention to this thread. I hadn't been
watchin
On Sun, Mar 4, 2012 at 9:59 AM, Simon Riggs wrote:
> On Sun, Mar 4, 2012 at 2:28 AM, Marti Raudsepp wrote:
>> On Sat, Mar 3, 2012 at 14:53, Simon Riggs wrote:
>>> Thanks Noah for drawing attention to this thread. I hadn't been
>>> watching. As you say, this work would allow me to freeze rows at
On Sun, Mar 4, 2012 at 2:28 AM, Marti Raudsepp wrote:
> On Sat, Mar 3, 2012 at 14:53, Simon Riggs wrote:
>> Thanks Noah for drawing attention to this thread. I hadn't been
>> watching. As you say, this work would allow me to freeze rows at load
>> time and avoid the overhead of hint bit setting,
On Sat, Mar 3, 2012 at 14:53, Simon Riggs wrote:
> Thanks Noah for drawing attention to this thread. I hadn't been
> watching. As you say, this work would allow me to freeze rows at load
> time and avoid the overhead of hint bit setting, which avoids
> performance issues from hint bit setting in c
On Sat, Feb 11, 2012 at 4:46 AM, Noah Misch wrote:
>> But I have to admit I'm intrigued by the idea of extending this to
>> other cases, if there's a reasonable way to do that. For example, if
>> we could fix things up so that we don't see a table at all if it was
>> created after we took our sn
Robert Haas wrote:
> Kevin Grittner wrote:
>> Well, personally I have a hard time calling READ COMMITTED
>> behavior sensible. Consider this:
>>
>> [ gigantic pile of fail ]
>
> Yeah, that's bad all right. I think it's hard to argue that the
> current behavior is sensible; the trick is to fig
On Mon, Feb 13, 2012 at 10:48 AM, Kevin Grittner
wrote:
> Well, personally I have a hard time calling READ COMMITTED behavior
> sensible. Consider this:
>
> [ gigantic pile of fail ]
Yeah, that's bad all right. I think it's hard to argue that the
current behavior is sensible; the trick is to fi
Robert Haas wrote:
> The example that I remember was related to SELECT FOR
> UPDATE/SELECT FOR SHARE. The idea of those statements is that you
> want to prevent the row from being updated or deleted until some
> other concurrent action is complete; for example, in the case of a
> foreign key, w
On Fri, Feb 10, 2012 at 11:46 PM, Noah Misch wrote:
> On Fri, Feb 10, 2012 at 01:59:18PM -0500, Robert Haas wrote:
>> On Fri, Feb 10, 2012 at 6:42 AM, Noah Misch wrote:
>> > I like the design you have chosen. ?It would find applications beyond
>> > TRUNCATE, so your use of non-specific naming is
On Fri, Feb 10, 2012 at 01:59:18PM -0500, Robert Haas wrote:
> I guess I'm not particularly excited by the idea of trying to make
> TRUNCATE MVCC-safe. I notice that the example involves the REPEATABLE
> READ isolation level, which is already known to be busted in a variety
> of ways; that's why w
On Fri, Feb 10, 2012 at 01:59:18PM -0500, Robert Haas wrote:
> On Fri, Feb 10, 2012 at 6:42 AM, Noah Misch wrote:
> > I like the design you have chosen. ?It would find applications beyond
> > TRUNCATE, so your use of non-specific naming is sound. ?For example, older
> > snapshots will see an empty
On Fri, Feb 10, 2012 at 6:42 AM, Noah Misch wrote:
> I like the design you have chosen. It would find applications beyond
> TRUNCATE, so your use of non-specific naming is sound. For example, older
> snapshots will see an empty table "t" after "CREATE TABLE t AS SELECT 1"
> commits; that's a com
On Thu, Feb 09, 2012 at 11:11:16PM +0200, Marti Raudsepp wrote:
> I've always been a little wary of using the TRUNCATE command due to
> the warning in the documentation about it not being "MVCC-safe":
> queries may silently give wrong results and it's hard to tell when
> they are affected.
>
> Tha
Hi!
I've always been a little wary of using the TRUNCATE command due to
the warning in the documentation about it not being "MVCC-safe":
queries may silently give wrong results and it's hard to tell when
they are affected.
That got me thinking, why can't we handle this like a standby server
does
37 matches
Mail list logo