Heikki Linnakangas napsal(a):
Zdenek Kotala wrote:
Heikki Linnakangas napsal(a):
Zdenek Kotala wrote:
Just a very quick look on your patch. See my comments:
...
2) PG_PAGE_LAYOUT_VERSION should be bump
The patch doesn't change the page layout AFAICS.
It is good question what is and what is not page layout. I think that
toast implementation is a member of page layout. OK it is called page
layout but better name should be On Disk Format (ODF). You will not
able to read 8.3 toasted table in 8.4.
It's clearly just a catalog change; the number and meaning of attributes
has changed, and that's reflected in CATALOG_VERSION_NO.
By by opinion it is not only catalog change. Probably you are right that it is
not part of page layout version. However, It changed column meaning on data
tables. You need to convert whole toast table and reindex toast table indext. It
is something what you cannot do online or you can but you need to exclusive lock
on toast table.
We need to be pragmatic, though, and think about how the conversion
would work, and if the version number change would help or hurt that
process. I'm not clear how we would handle the toast table change. If
we're going to handle it by retoasting all attributes when the main heap
page is read in, then I suppose we'd actually change the version number
of the *heap* page, not toast table pages, when the heap page is
retoasted. However, if you want to do it toast-page at a time, or
toast-tuple at a time, you can just look at the number of attributes on
the toast tuple to determine which format it's in.
I'm trying to write down a toast conversion concept. It looks like that
it is more complex that I expected.
Note that bumping the version number is not free. We haven't made any
changes in 8.4 this far that would require bumping it. If we do bump it,
the next version with online-upgrade support will need to deal with it,
if only to increment and write back the page.
Yes, I know about it. But I'm afraid that 8.3->8.4 in-place upgrade will not
work.
Zdenek
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
