Re: [PATCH] ltree hash functions

From: Tommy Pavlicek <tommypav122(at)gmail(dot)com>
To: jian he <jian(dot)universality(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-12-01 00:44:40
Message-ID: CAEhP-W8_N3seEhQVDhTxV17qn3qZsN6-xhc=Qn=AwLn+WmKWPA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Nov 28, 2023 at 7:38 PM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
> you only change Makefile, you also need to change contrib/ltree/meson.build?
> Do you need to use EXPLAIN to demo the index usage?

Thanks! Yes, I missed the Meson build file. I added additional
commands with EXPLAIN (COSTS OFF) as I found in other places.

Patch updated for those comments (and a touch of cleanup in the tests) attached.

Attachment Content-Type Size
0001-ltree-hash-v3.patch application/octet-stream 10.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2023-12-01 01:30:41 Re: Something seems weird inside tts_virtual_copyslot()
Previous Message Andres Freund 2023-12-01 00:31:38 Re: PostgreSql: Canceled on conflict out to old pivot