From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | tgl(at)sss(dot)pgh(dot)pa(dot)us (Tom Lane) |
Cc: | hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] samekeys |
Date: | 1999-02-09 16:35:20 |
Message-ID: | 199902091635.LAA20398@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> writes:
> > This basically says that key1, which is the old key, has to match key2
> > for the length of key1. If key2 has extra keys after that, that is
> > fine. We will still consider the keys equal. The old code obviously
> > was broken and badly thought out.
> > ...
> > I am unsure if samekeys should just test the first key for equality, or
> > the full length of key1 as I have done.
>
> The comment in front of samekeys claimed:
>
> * It isn't necessary to check that each sublist exactly contain
> * the same elements because if the routine that built these
> * sublists together is correct, having one element in common
> * implies having all elements in common.
>
> Was that wrong? Or, perhaps, it was once right but no longer?
> It sounded like fragile coding to me, but I didn't have reason
> to know it was broken...
I think it was wrong. It clearly was not passing the right parameters.
As far as I know (1,2,3) and (3,2,1) are not the same. Their test would
just take '1' and see if it is in (3,2,1).
--
Bruce Momjian | http://www.op.net/~candle
maillist(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
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 1999-02-09 16:38:19 | Re: [HACKERS] v6.4.3 ? |
Previous Message | Tom Lane | 1999-02-09 16:26:01 | Re: [HACKERS] v6.4.3 ? |