Re: [HACKERS] Re: Tuple-valued datums on Alpha (was Re: 7.1 on DEC/Alpha)

From: Ryan Kirkpatrick <pgsql(at)rkirkpat(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Brent Verner <brent(at)rcfile(dot)org>, pgsql-patches(at)postgreSQL(dot)org
Subject: Re: [HACKERS] Re: Tuple-valued datums on Alpha (was Re: 7.1 on DEC/Alpha)
Date: 2000-12-29 01:55:30
Message-ID: Pine.LNX.4.10.10012281849590.8458-100000@excelsior.rkirkpat.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Wed, 27 Dec 2000, Tom Lane wrote:

> After further study, I realized that fetchatt() and a number of other
> places were not prepared to cope with 8-byte pass-by-value datatypes.
> Most of them weren't checking for cases they couldn't handle, either.
>
> Here is a revised patch for you to try (this includes yesterday's patch
> plus more changes, so you'll need to reverse out the prior patch before
> applying this one). NOTE you will need to do a full reconfigure and
> rebuild to make this fly --- I'd suggest "make distclean" to start.

Good news is that it solves the 'misc' regression test failure. It
now passes with flying colors! The bad news is that the 'oid' regression
test is still broken (with the exact same problem as before). I think
Brent hit the same problem... I guess, verify that your oid fix actually
hit the CVS tree, and if it did, rethink the solution. :(
For testing I used the snapshot from ftp.postgresql.org:/pub/dev/
dated yesterday on my Alpha XLT366 running Debian GNU/Linux 2.2r0, kernel
2.2.17. Though I found the 'configure' file actually a copy of 'config.in'
and had to run the latter file through autoconf to get the correct version
of the former file. Weird.
Also, I tested a patches source tree on an Linux/x86 box, and it
passed all regression tests w/o problems. I can test the patched source
tree on a Linux/Sparc machine if you want (bit more effort required to do
so).
Overall, it looks like we are making progress! Thanks to both you
and Brent for looking deeper into these problems. TTYL.

---------------------------------------------------------------------------
| "For to me to live is Christ, and to die is gain." |
| --- Philippians 1:21 (KJV) |
---------------------------------------------------------------------------
| Ryan Kirkpatrick | Boulder, Colorado | http://www.rkirkpat.net/ |
---------------------------------------------------------------------------

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Brent Verner 2000-12-29 02:45:56 Re: Alpha tas() patch
Previous Message Nathan Myers 2000-12-29 00:10:30 Re: Assuming that TAS() will succeed the first time is verboten

Browse pgsql-patches by date

  From Date Subject
Next Message Brent Verner 2000-12-29 02:45:56 Re: Alpha tas() patch
Previous Message Bruce Momjian 2000-12-28 23:56:58 Re: Two minor fixes to reduce resource usage in JDBC drivers