We are about to submit brand bew contrib/ltree module
(first draft of documetation is available from
http://www.sai.msu.su/~megera/postgres/gist/ltree/)
and I have a question what version to submit - 7.2 or 7.3 ?
We have tested the module with 7.2 and have a patch for 7.3.
Probably better to submit version for 7.3 development and
have archive for 7.2.

	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


Sun, 21 Jul 2002 18:38:16 +0300 (GMT)
Oleg Bartunov <oleg@sai.msu.su>
contrib/ltree for 7.2 or 7.3 ?
Oleg Bartunov <oleg@sai.msu.su> writes:
> We are about to submit brand bew contrib/ltree module
> (first draft of documetation is available from
> http://www.sai.msu.su/~megera/postgres/gist/ltree/)
> and I have a question what version to submit - 7.2 or 7.3 ?

7.3.  There are unlikely to be any more 7.2 releases, and in any
case they would be bugfixes only, no new features.

			regards, tom lane

Sun, 21 Jul 2002 11:55:14 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: contrib/ltree for 7.2 or 7.3 ? 
On Sun, 21 Jul 2002, Tom Lane wrote:

> Oleg Bartunov <oleg@sai.msu.su> writes:
> > We are about to submit brand bew contrib/ltree module
> > (first draft of documetation is available from
> > http://www.sai.msu.su/~megera/postgres/gist/ltree/)
> > and I have a question what version to submit - 7.2 or 7.3 ?
>
> 7.3.  There are unlikely to be any more 7.2 releases, and in any
> case they would be bugfixes only, no new features.

OK. We've got documentation written and the module could be downloaded
from http://www.sai.msu.su/~megera/postgres/gist/ltree/ltree.tar.gz
It's works with current CVS and there is patch.72 within the archive,
so people could use it with 7.2 release.

I've attached text version of documentation, it's about 16Kb, sorry for that.
HTML version is available from http://www.sai.msu.su/~megera/postgres/gist/ltree/

Also, we prepared test data based on DMOZ catalog (about 300,000 nodes)
and encourage people to play with queries.

One known issue: It'll not works with 64-bit OS. We'll certainly fix this
but will appreciate if somebody with access to 64-bit machine could help us.
It's known problem with byte-alignment.


>
> 			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
Tue, 23 Jul 2002 19:33:21 +0300 (GMT)
Oleg Bartunov <oleg@sai.msu.su>
Re: contrib/ltree for 7.2 or 7.3 ? 

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.

---------------------------------------------------------------------------


Oleg Bartunov wrote:
> On Sun, 21 Jul 2002, Tom Lane wrote:
> 
> > Oleg Bartunov <oleg@sai.msu.su> writes:
> > > We are about to submit brand bew contrib/ltree module
> > > (first draft of documetation is available from
> > > http://www.sai.msu.su/~megera/postgres/gist/ltree/)
> > > and I have a question what version to submit - 7.2 or 7.3 ?
> >
> > 7.3.  There are unlikely to be any more 7.2 releases, and in any
> > case they would be bugfixes only, no new features.
> 
> OK. We've got documentation written and the module could be downloaded
> from http://www.sai.msu.su/~megera/postgres/gist/ltree/ltree.tar.gz
> It's works with current CVS and there is patch.72 within the archive,
> so people could use it with 7.2 release.
> 
> I've attached text version of documentation, it's about 16Kb, sorry for that.
> HTML version is available from http://www.sai.msu.su/~megera/postgres/gist/ltree/
> 
> Also, we prepared test data based on DMOZ catalog (about 300,000 nodes)
> and encourage people to play with queries.
> 
> One known issue: It'll not works with 64-bit OS. We'll certainly fix this
> but will appreciate if somebody with access to 64-bit machine could help us.
> It's known problem with byte-alignment.
> 
> 
> >
> > 			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

Content-Description: 

[ Attachment, skipping... ]

-- 
  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

Tue, 23 Jul 2002 19:13:18 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: contrib/ltree for 7.2 or 7.3 ?

Patch applied.  Thanks.

---------------------------------------------------------------------------




Oleg Bartunov wrote:
> On Sun, 21 Jul 2002, Tom Lane wrote:
> 
> > Oleg Bartunov <oleg@sai.msu.su> writes:
> > > We are about to submit brand bew contrib/ltree module
> > > (first draft of documetation is available from
> > > http://www.sai.msu.su/~megera/postgres/gist/ltree/)
> > > and I have a question what version to submit - 7.2 or 7.3 ?
> >
> > 7.3.  There are unlikely to be any more 7.2 releases, and in any
> > case they would be bugfixes only, no new features.
> 
> OK. We've got documentation written and the module could be downloaded
> from http://www.sai.msu.su/~megera/postgres/gist/ltree/ltree.tar.gz
> It's works with current CVS and there is patch.72 within the archive,
> so people could use it with 7.2 release.
> 
> I've attached text version of documentation, it's about 16Kb, sorry for that.
> HTML version is available from http://www.sai.msu.su/~megera/postgres/gist/ltree/
> 
> Also, we prepared test data based on DMOZ catalog (about 300,000 nodes)
> and encourage people to play with queries.
> 
> One known issue: It'll not works with 64-bit OS. We'll certainly fix this
> but will appreciate if somebody with access to 64-bit machine could help us.
> It's known problem with byte-alignment.
> 
> 
> >
> > 			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

Content-Description: 

[ Attachment, skipping... ]

-- 
  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

Tue, 30 Jul 2002 12:44:26 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: contrib/ltree for 7.2 or 7.3 ?
> Oleg Bartunov wrote:
>> One known issue: It'll not works with 64-bit OS. We'll certainly fix this
>> but will appreciate if somebody with access to 64-bit machine could help us.

Actually, it dumps core instantly on 32-bit machines too, if they are
pickier about alignment than Intel hardware is.  You can't map
structures onto char[] arrays that start at odd byte offsets and not
expect trouble.

I also do not trust macros like this:

typedef struct {
	int32	len;
	uint16	numlevel;
	char	data[1];
} ltree;

#define LTREE_HDRSIZE	( sizeof(int32) + sizeof(uint16) )

because they take no account of the possibility of padding between fields.

			regards, tom lane

Tue, 30 Jul 2002 14:48:04 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: contrib/ltree for 7.2 or 7.3 ? 
I'm now working on it.

Tom Lane wrote:
>>Oleg Bartunov wrote:
>>
>>>One known issue: It'll not works with 64-bit OS. We'll certainly fix this
>>>but will appreciate if somebody with access to 64-bit machine could help us.
>>
> 
> Actually, it dumps core instantly on 32-bit machines too, if they are
> pickier about alignment than Intel hardware is.  You can't map
> structures onto char[] arrays that start at odd byte offsets and not
> expect trouble.
> 
> I also do not trust macros like this:
> 
> typedef struct {
> 	int32	len;
> 	uint16	numlevel;
> 	char	data[1];
> } ltree;
> 
> #define LTREE_HDRSIZE	( sizeof(int32) + sizeof(uint16) )
> 
> because they take no account of the possibility of padding between fields.
> 
> 			regards, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 2: you can get off all lists at once with the unregister command
>     (send "unregister YourEmailAddressHere" to majordomo@postgresql.org)
> 


-- 
Teodor Sigaev
teodor@stack.net



Tue, 30 Jul 2002 22:52:11 +0400
Teodor Sigaev <teodor@stack.net>
Re: contrib/ltree for 7.2 or 7.3 ?

Folks, has this been fixed?

---------------------------------------------------------------------------

Tom Lane wrote:
> > Oleg Bartunov wrote:
> >> One known issue: It'll not works with 64-bit OS. We'll certainly fix this
> >> but will appreciate if somebody with access to 64-bit machine could help us.
> 
> Actually, it dumps core instantly on 32-bit machines too, if they are
> pickier about alignment than Intel hardware is.  You can't map
> structures onto char[] arrays that start at odd byte offsets and not
> expect trouble.
> 
> I also do not trust macros like this:
> 
> typedef struct {
> 	int32	len;
> 	uint16	numlevel;
> 	char	data[1];
> } ltree;
> 
> #define LTREE_HDRSIZE	( sizeof(int32) + sizeof(uint16) )
> 
> because they take no account of the possibility of padding between fields.
> 
> 			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

Tue, 6 Aug 2002 02:27:44 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: contrib/ltree for 7.2 or 7.3 ?
On Tue, 6 Aug 2002, Bruce Momjian wrote:

>
> Folks, has this been fixed?
>

Bruce, you already did apply patches which fixed this issue :-)


> ---------------------------------------------------------------------------
>
> Tom Lane wrote:
> > > Oleg Bartunov wrote:
> > >> One known issue: It'll not works with 64-bit OS. We'll certainly fix this
> > >> but will appreciate if somebody with access to 64-bit machine could help us.
> >
> > Actually, it dumps core instantly on 32-bit machines too, if they are
> > pickier about alignment than Intel hardware is.  You can't map
> > structures onto char[] arrays that start at odd byte offsets and not
> > expect trouble.
> >
> > I also do not trust macros like this:
> >
> > typedef struct {
> > 	int32	len;
> > 	uint16	numlevel;
> > 	char	data[1];
> > } ltree;
> >
> > #define LTREE_HDRSIZE	( sizeof(int32) + sizeof(uint16) )
> >
> > because they take no account of the possibility of padding between fields.
> >
> > 			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


Tue, 6 Aug 2002 11:54:35 +0300 (GMT)
Oleg Bartunov <oleg@sai.msu.su>
Re: contrib/ltree for 7.2 or 7.3 ?
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Folks, has this been fixed?

I can report that ltree passes its regression test now on HPPA.
It didn't before...

			regards, tom lane

Tue, 06 Aug 2002 09:40:40 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: contrib/ltree for 7.2 or 7.3 ? 