Re: [HACKERS] five-key syscaches

2010-07-14 Thread Robert Haas
On Wed, Jul 14, 2010 at 10:56 AM, Yeb Havinga wrote: > Robert Haas wrote: >> On Wed, Jul 14, 2010 at 7:27 AM, Yeb Havinga wrote: >>> Attach is a new patch with all things described above addressed. >> Thanks! >> >> I think we should probably hold off applying this until some of the >> other KNNGI

Re: [HACKERS] five-key syscaches

2010-07-14 Thread Yeb Havinga
Robert Haas wrote: On Wed, Jul 14, 2010 at 7:27 AM, Yeb Havinga wrote: Attach is a new patch with all things described above addressed. Thanks! I think we should probably hold off applying this until some of the other KNNGIST work is ready, or we have some other concrete need for 5-k

Re: [HACKERS] five-key syscaches

2010-07-14 Thread Robert Haas
On Wed, Jul 14, 2010 at 7:27 AM, Yeb Havinga wrote: > Attach is a new patch with all things described above addressed. Thanks! I think we should probably hold off applying this until some of the other KNNGIST work is ready, or we have some other concrete need for 5-key syscaches. -- Robert Haa

Re: [HACKERS] five-key syscaches

2010-07-14 Thread Yeb Havinga
Hello Robert, As part of the current reviewfest, I reviewed your patch, and made some changes on the way. This was all ok: *) while proofreading I did not find typos other than the one that Joachim had already pointed out. *) the addition of 5-key lookups to the existing ones seems a natural

Re: [HACKERS] five-key syscaches

2010-03-29 Thread Robert Haas
On Mon, Mar 29, 2010 at 4:21 AM, Joachim Wieland wrote: > On Mon, Mar 29, 2010 at 12:32 AM, Robert Haas wrote: >> Per previous discussion, PFA a patch to change the maximum number of >> keys for a syscache from 4 to 5. >> >> http://archives.postgresql.org/pgsql-hackers/2010-02/msg01105.php >> >>

Re: [HACKERS] five-key syscaches

2010-03-29 Thread Joachim Wieland
On Mon, Mar 29, 2010 at 12:32 AM, Robert Haas wrote: > Per previous discussion, PFA a patch to change the maximum number of > keys for a syscache from 4 to 5. > > http://archives.postgresql.org/pgsql-hackers/2010-02/msg01105.php > > This is intended for application to 9.1, and is supporting > infr

[HACKERS] five-key syscaches

2010-03-28 Thread Robert Haas
Per previous discussion, PFA a patch to change the maximum number of keys for a syscache from 4 to 5. http://archives.postgresql.org/pgsql-hackers/2010-02/msg01105.php This is intended for application to 9.1, and is supporting infrastructure for knngist. ...Robert syscache5.patch Description: