Hi,

we finished first stage of our proposal for changing of index AM tables
(see for reference http://fts.postgresql.org/db/mw/msg.html?mid=1029290)

I attached 3 files:

1. patch_72_systbl.gz - patch to current CVS
2. btree_gist.tar.gz  - contrib/btree_gist module -
                        implementation of Btree using GiST with
                        support of int4 and timestamp types.
3. test.tar.gz        - test suite for brave (not for applying !)

Regression tests and our tests passed fine
Patch is for today CVS, please apply them asap to avoid possible
conflicts.

Now we're going to 2nd stage of our proposal.
We plan to remove pg_index.indislossy (now we have pg_amop.amopreqcheck)
and  pg_index.indhaskeytype (it's just don't used, all functionality
                             is in pg_opclass.opckeytype now)

question:

Do we need to normalize pg_amop and pg_amproc tables ?
Tom was concerned (http://fts.postgresql.org/db/mw/msg.html?mid=1025860)
about possible performance degradation.

We think it's possible to leave tables as is. Of course it'd require
some attention when updating these tables.


	Regards,
		Oleg
_____________________________________________________________
Oleg Bartunov, sci.researcher, hostmaster of AstroNet,
Sternberg Astronomical Institute, Moscow University (Russia)
Internet: oleg@sai.msu.su, http://www.sai.msu.su/~megera/
phone: +007(095)939-16-83, +007(095)939-23-83
Wed, 15 Aug 2001 15:07:20 +0300 (GMT)
Oleg Bartunov <oleg@sai.msu.su>
Patches (current CVS) for changes if index AM tables

Your patch has been added to the PostgreSQL unapplied patches list at:

	http://candle.pha.pa.us/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

> Hi,
> 
> we finished first stage of our proposal for changing of index AM tables
> (see for reference http://fts.postgresql.org/db/mw/msg.html?mid=1029290)
> 
> I attached 3 files:
> 
> 1. patch_72_systbl.gz - patch to current CVS
> 2. btree_gist.tar.gz  - contrib/btree_gist module -
>                         implementation of Btree using GiST with
>                         support of int4 and timestamp types.
> 3. test.tar.gz        - test suite for brave (not for applying !)
> 
> Regression tests and our tests passed fine
> Patch is for today CVS, please apply them asap to avoid possible
> conflicts.
> 
> Now we're going to 2nd stage of our proposal.
> We plan to remove pg_index.indislossy (now we have pg_amop.amopreqcheck)
> and  pg_index.indhaskeytype (it's just don't used, all functionality
>                              is in pg_opclass.opckeytype now)
> 
> question:
> 
> Do we need to normalize pg_amop and pg_amproc tables ?
> Tom was concerned (http://fts.postgresql.org/db/mw/msg.html?mid=1025860)
> about possible performance degradation.
> 
> We think it's possible to leave tables as is. Of course it'd require
> some attention when updating these tables.
> 
> 
> 	Regards,
> 		Oleg
> _____________________________________________________________
> Oleg Bartunov, sci.researcher, hostmaster of AstroNet,
> Sternberg Astronomical Institute, Moscow University (Russia)
> Internet: oleg@sai.msu.su, http://www.sai.msu.su/~megera/
> phone: +007(095)939-16-83, +007(095)939-23-83

Content-Description: 

[ Attachment, skipping... ]

Content-Description: 

[ Attachment, skipping... ]

Content-Description: 

[ Attachment, skipping... ]

> 
> ---------------------------(end of broadcast)---------------------------
> TIP 2: you can get off all lists at once with the unregister command
>     (send "unregister YourEmailAddressHere" to majordomo@postgresql.org)

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Thu, 16 Aug 2001 12:39:41 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: Patches (current CVS) for changes if index AM tables
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> I will try to apply it within the next 48 hours.

I will take responsibility for reviewing and applying this patch.  In a
quick lookover, I saw a few adjustments I wanted to make...

			regards, tom lane

Thu, 16 Aug 2001 13:04:00 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: Patches (current CVS) for changes if index AM tables 

Got it.

> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > I will try to apply it within the next 48 hours.
> 
> I will take responsibility for reviewing and applying this patch.  In a
> quick lookover, I saw a few adjustments I wanted to make...
> 
> 			regards, tom lane
> 

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Thu, 16 Aug 2001 13:04:47 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: Patches (current CVS) for changes if index AM tables
On Thu, 16 Aug 2001, Tom Lane wrote:

> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > I will try to apply it within the next 48 hours.
>
> I will take responsibility for reviewing and applying this patch.  In a
> quick lookover, I saw a few adjustments I wanted to make...

thanks, please know we prepared a patch for contrib/btree_gist. It fixes
memory leak and improve performance. Should I sent it right now or
wait until you apply previous patches.

>
> 			regards, tom lane
>

	Regards,
		Oleg
_____________________________________________________________
Oleg Bartunov, sci.researcher, hostmaster of AstroNet,
Sternberg Astronomical Institute, Moscow University (Russia)
Internet: oleg@sai.msu.su, http://www.sai.msu.su/~megera/
phone: +007(095)939-16-83, +007(095)939-23-83


Thu, 16 Aug 2001 20:36:56 +0300 (GMT)
Oleg Bartunov <oleg@sai.msu.su>
Re: Patches (current CVS) for changes if index AM tables
Oleg Bartunov <oleg@sai.msu.su> writes:
> thanks, please know we prepared a patch for contrib/btree_gist. It fixes
> memory leak and improve performance. Should I sent it right now or
> wait until you apply previous patches.

I haven't touched that contrib code yet at all.  Why don't you just
re-wrap that tarball with the updates and resubmit it?

			regards, tom lane

Thu, 16 Aug 2001 13:47:05 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: Patches (current CVS) for changes if index AM tables 
On Thu, 16 Aug 2001, Tom Lane wrote:

> Oleg Bartunov <oleg@sai.msu.su> writes:
> > thanks, please know we prepared a patch for contrib/btree_gist. It fixes
> > memory leak and improve performance. Should I sent it right now or
> > wait until you apply previous patches.
>
> I haven't touched that contrib code yet at all.  Why don't you just
> re-wrap that tarball with the updates and resubmit it?

ok. I attached btree_gist.tar.gz which should be used instead of
older one.


>
> 			regards, tom lane
>

	Regards,
		Oleg
_____________________________________________________________
Oleg Bartunov, sci.researcher, hostmaster of AstroNet,
Sternberg Astronomical Institute, Moscow University (Russia)
Internet: oleg@sai.msu.su, http://www.sai.msu.su/~megera/
phone: +007(095)939-16-83, +007(095)939-23-83
Thu, 16 Aug 2001 20:58:51 +0300 (GMT)
Oleg Bartunov <oleg@sai.msu.su>
Re: Patches (current CVS) for changes if index AM tables