Re: Concrete proposal for large objects and MVCC

From: Tatsuo Ishii <t-ishii(at)sra(dot)co(dot)jp>
To: tgl(at)sss(dot)pgh(dot)pa(dot)us
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Concrete proposal for large objects and MVCC
Date: 2005-06-10 16:56:25
Message-ID: 20050611.015625.104031633.t-ishii@sra.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> I spent a little bit of time thinking about what it would mean exactly
> for large-object operations to obey MVCC, and decided that there are
> more worms in that can than I had realized. Part of the problem is
> that we have no concept of a lock on an individual LO, and thus
> operations that really should be serialized, such as DROP, aren't going
> to work very well. We could implement DROP as the equivalent of
> DELETE FROM pg_largeobject WHERE loid = nnn;
> with an MVCC snapshot --- but there is no guarantee that we won't miss
> a page that someone else is concurrently inserting into that same large
> object.
>
> So what I'm thinking is that the prudent course is to leave writing
> semantics as they are, namely SnapshotNow rules. (What this means
> in practice is you get "tuple concurrently updated" errors if two
> transactions try to write the same page of the same LO concurrently.
> We have seen few if any complaints about that error in connection with
> LO operations, so ISTM there's not a problem there that needs solving.)
>
> The problem we do need to solve is letting pg_dump have a stable view
> of the database's large objects. I propose that we can fix this in
> a suitably narrow way by making the following definition:
>
> * A large object descriptor opened for read-only access saves
> the current ActiveSnapshot and uses that snapshot to read
> pg_largeobject for the duration of its existence.
>
> * A large object descriptor opened for write-only or read-write
> access uses SnapshotNow, same as before.
>
> This avoids the risk of creating any serious backwards-compatibility
> issues: if there's anyone out there who does need SnapshotNow reads,
> they just have to be sure to open the LO in read-write mode to have
> fully backward compatible operation.
>
> Comments, objections?

Besides the MVCC issue, I am not sure it's a good idea LO being binded
to OID. In my understanding OID is solely used to distinguish each LO
in a database. In another word, it's just a key to LO. I think giving
explicit key when creating a LO has some benefits:

1) not need to worry about OID wrap around problem
2) easier to find orpahn LO
3) for replication systems it's easier to replicate LOs

What do you think?
--
Tatsuo Ishii

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Joshua D. Drake 2005-06-10 17:03:25 Re: Concrete proposal for large objects and MVCC
Previous Message Tom Lane 2005-06-10 16:46:32 Re: Gist Recovery testing