Skip site navigation (1) Skip section navigation (2)

Re: [GENERAL] Some Solaris notes, and an invitation

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Sullivan <andrew(at)libertyrms(dot)info>,PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>,justin(at)postgresql(dot)org
Subject: Re: [GENERAL] Some Solaris notes, and an invitation
Date: 2002-07-19 18:43:30
Message-ID: 200207191843.g6JIhVS05281@candle.pha.pa.us (view raw or flat)
Thread:
Lists: pgsql-generalpgsql-patches
Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > The following diff adds the NetBSD qsort() for all Solaris platforms. 
> 
> Do we know that Solaris is the only platform with the problem?
> 
> In the back of my mind is the thought that if we used this qsort on
> *all* platforms, we'd eliminate some platform-to-platform sort ordering
> differences that occasionally cause regression test problems.  And I
> see little downside to using it everywhere.

That's a tough call. I am hesitant to replace qsort on platforms where
it is known to be working.  If an OS has some platform-specific
optimized qsort, our replacing it could cause problems.  Not sure how
likely that is to have happened, but Solaris clearly broke theirs so
someone must be modifying libc qsort somewhere.  In fact, some qsorts
are not even quicksort, so I tend to defer to the libc authors to
provide the best qsort for their platform.  In fact, this email suggests
there is quite a bit of variation in qsort implementations:
	
	http://www.dietlibc.org/mailinglists/dietlibc/2001/12-2001/msg00024.html

I don't think we can replace library functions unless we know they are
broken or missing, or if several people using the OS agree with the
replacement, as they have on Solaris.

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

In response to

pgsql-patches by date

Next:From: Neil ConwayDate: 2002-07-19 20:17:52
Subject: Re: prepareable statements
Previous:From: Andrew SullivanDate: 2002-07-19 18:42:57
Subject: Re: [GENERAL] Some Solaris notes, and an invitation

pgsql-general by date

Next:From: Mike MascariDate: 2002-07-19 19:15:05
Subject: Re: Multi-Versions and Vacuum -- cf Oracle & Vacuum alt
Previous:From: Andrew SullivanDate: 2002-07-19 18:42:57
Subject: Re: [GENERAL] Some Solaris notes, and an invitation

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group