[HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Zdenek Kotala
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 implementati

Re: [HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Zdenek Kotala
Alvaro Herrera napsal(a): Zdenek Kotala wrote: If I'm thinking more, it is not probably CATALOG_VERSION_NO as well. Because toast table is created on demand. It is not in BKI. It's not catversion in the sense that there's no catalog change, but it certainly requires a catversion bump due to i

Re: [HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Tom Lane
Alvaro Herrera <[EMAIL PROTECTED]> writes: > Zdenek Kotala wrote: >> If I'm thinking more, it is not probably CATALOG_VERSION_NO as well. >> Because toast table is created on demand. It is not in BKI. > It's not catversion in the sense that there's no catalog change, but it > certainly requires a

Re: [HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Alvaro Herrera
Zdenek Kotala wrote: > If I'm thinking more, it is not probably CATALOG_VERSION_NO as well. > Because toast table is created on demand. It is not in BKI. It's not catversion in the sense that there's no catalog change, but it certainly requires a catversion bump due to internal changes. Otherwis

[HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Zdenek Kotala
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

[HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Zdenek Kotala
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

[HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Heikki Linnakangas
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 th

[HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Heikki Linnakangas
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. -- Heikki Linnakangas EnterpriseDB http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hacker

[HACKERS] Re: toast by chunk-end (was Re: PG_PAGE_LAYOUT_VERSION 5 - time for change)

2008-11-18 Thread Zdenek Kotala
Alvaro Herrera napsal(a): Zdenek Kotala wrote: Alvaro Herrera napsal(a): Heikki Linnakangas wrote: Hmm, you're right. I think it can be made to work by storing the *end* offset of each chunk. To find the chunk containing offset X, search for the first chunk with end_offset > X. FWIW I'm t