Re: - WIP Patch Updatable Cursor

From: "John Bartlett" <johnb(at)fast(dot)fujitsu(dot)com(dot)au>
To: <pgsql-hackers(at)postgresql(dot)org>, <pgsql-patches(at)postgresql(dot)org>
Cc: "'Heikki Linnakangas'" <heikki(at)enterprisedb(dot)com>
Subject: Re: - WIP Patch Updatable Cursor
Date: 2007-02-28 04:14:54
Message-ID: !~!UENERkVCMDkAAQACAAAAAAAAAAAAAAAAABgAAAAAAAAA/w1P36NVXkuUGCDGiycCfMKAAAAQAAAAOYTSnVrqlkKlNDJ5Wv6uogEAAAAA@fast.fujitsu.com.au
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Hi,

A list of ctids is stored in the file.

The file is used to store the ctids during an updatable cursor transaction.

It is set up as a permanent file as it has a potential lifetime of
preserving data between crashes of the backend. Temporary files tend to be
used for data that is defined within a single command. In this case the file
needs to exist within a transaction and across backend processes.

The file gram.y has been corrected in my version.

The files ctidListStore.c and ctidListStore.h were pasted into the patch
file, as the diff -N command produced a file of several hundred thousand
lines.

Regards,
John Bartlett

-----Original Message-----
From: Heikki Linnakangas [mailto:hlinnaka(at)gmail(dot)com] On Behalf Of Heikki
Linnakangas
Sent: Tuesday, 27 February 2007 10:03 PM
To: John Bartlett
Cc: pgsql-patches(at)postgresql(dot)org; pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES]

John Bartlett wrote:
> The community may wish to comment on the following issue:
>
> 1) At present the file that will contain the list of ctids is going
into
> a new directory called pg_ctids, analogous to pg_twophase, and also stored
> in the pg_data directory.

I don't understand this. What's stored in the file and why? If they're
only needed within the transaction, surely a temp file would be more
appropriate?

The new ctidListStore.c file in the patch is not in a valid diff-format.
I also noticed that you've moved the line beginning with "CREATE_ROLE"
in gram.y so that it's not in alphabetical order anymore.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

This is an email from Fujitsu Australia Software Technology Pty Ltd, ABN 27 003 693 481. It is confidential to the ordinary user of the email address to which it was addressed and may contain copyright and/or legally privileged information. No one else may read, print, store, copy or forward all or any of it or its attachments. If you receive this email in error, please return to sender. Thank you.

If you do not wish to receive commercial email messages from Fujitsu Australia Software Technology Pty Ltd, please email unsubscribe(at)fast(dot)fujitsu(dot)com(dot)au

In response to

  • Re: at 2007-02-27 11:03:12 from Heikki Linnakangas

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Paul Silveira 2007-02-28 04:19:03 Re: Proposal for Implenting read-only queries during wal replay (SoC 2007)
Previous Message Tom Lane 2007-02-28 03:48:25 Re: Resumable vacuum proposal and design overview

Browse pgsql-patches by date

  From Date Subject
Next Message Gavin Sherry 2007-02-28 04:22:30 Re: [HACKERS] - WIP Patch Updatable Cursor
Previous Message Bruce Momjian 2007-02-28 03:03:58 Re: [HACKERS]