[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

[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

[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

[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

[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

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.

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

[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