| From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
|---|---|
| To: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
| Cc: | Alexander Korotkov <akorotkov(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: pgsql: Introduce hash_search_with_hash_value() function |
| Date: | 2024-08-08 04:44:31 |
| Message-ID: | CAFj8pRBka02SrWnu0iTCTCOfBUco_A4uLNjuZj3UH0CkVJMF6Q@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
st 7. 8. 2024 v 22:25 odesílatel Alexander Korotkov <aekorotkov(at)gmail(dot)com>
napsal:
> On Wed, Aug 7, 2024 at 1:34 PM Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
> wrote:
> > st 7. 8. 2024 v 12:22 odesílatel Alexander Korotkov <
> aekorotkov(at)gmail(dot)com> napsal:
> >> Thank you.
> >> Please see 40064a8ee1 takes special efforts to match HTAB hash
> >> function to syscache hash function. By default, their hash values
> >> don't match and you can't simply use syscache hash value to search
> >> HTAB. This probably should be mentioned in the header comment of
> >> hash_seq_init_with_hash_value().
> >
> >
> > yes, enhancing doc should be great + maybe assert
>
> Please check the patch, which adds a caveat to the function header
> comment. I don't particularly like an assert here, because there
> could be use-cases besides syscache callbacks, which could legally use
> default hash function.
>
looks well
Regards
Pavel
>
> ------
> Regards,
> Alexander Korotkov
> Supabase
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2024-08-08 06:07:55 | pgsql: Revert ECPG's use of pnstrdup() |
| Previous Message | Michael Paquier | 2024-08-07 23:15:44 | Re: pgsql: Fix more holes with SLRU code in need of int64 for segment numbe |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amit Kapila | 2024-08-08 04:52:56 | Re: Found issues related with logical replication and 2PC |
| Previous Message | Amit Kapila | 2024-08-08 04:23:26 | Re: Found issues related with logical replication and 2PC |