Re: [PATCH] ltree hash functions

From: jian he <jian(dot)universality(at)gmail(dot)com>
To: Tommy Pavlicek <tommypav122(at)gmail(dot)com>
Cc: daniel(at)yesql(dot)se, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] ltree hash functions
Date: 2023-11-29 01:37:55
Message-ID: CACJufxGd4_qSqU3-j9TFOS7JD0Jx+fSFNdVBnEeprP35ppFscQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Nov 29, 2023 at 6:09 AM Tommy Pavlicek <tommypav122(at)gmail(dot)com> wrote:
>
> On Thu, Jul 6, 2023 at 2:18 AM Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> >
> > > On 19 Jun 2023, at 11:18, Tommy Pavlicek <tommypav122(at)gmail(dot)com> wrote:
> >
> > > Tommy, are you interested in extending ALTER OPERATOR to allow this,
> > > which would also allow fixing the ltree operator?
> > >
> > > Yes, I can do that. I took a look over the code and email thread and it seems like it should be relatively straight forward. I'll put a patch together for that and then update this patch to alter the operator.
> >
> > Did you have a chance to look at this for an updated patch for this commitfest?
>
> I finally had a chance to look at this and I've updated the patch to
> alter the = operator to enable hash joins.
>
> This is ready to be looked at now.
>
> Is there anything I need to do to move this forward?
>

you only change Makefile, you also need to change contrib/ltree/meson.build?

+drop index tstidx;
+create index tstidx on ltreetest using hash (t);
+set enable_seqscan=off;
+
+SELECT * FROM ltreetest WHERE t = '12.3' order by t asc;

Do you need to use EXPLAIN to demo the index usage?

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2023-11-29 01:58:41 Re: remaining sql/json patches
Previous Message Melanie Plageman 2023-11-29 01:21:28 Re: Streaming I/O, vectored I/O (WIP)