From: | Mark Rofail <markm(dot)rofail(at)gmail(dot)com> |
---|---|
To: | David Steele <david(at)pgmasters(dot)net> |
Cc: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Andreas Karlsson <andreas(at)proxel(dot)se>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Erik Rijkers <er(at)xs4all(dot)nl>, Hans-Jürgen Schönig <hs(at)cybertec(dot)at>, Robert Haas <robertmhaas(at)gmail(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] GSoC 2017: Foreign Key Arrays |
Date: | 2018-04-10 13:57:17 |
Message-ID: | CAJvoCuvdSx8fizeFYLLf6xeJHG6mDF+z2vcuKKrD25RBahjX8Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello David,
On Tue, Apr 10, 2018 at 3:47 PM, David Steele <david(at)pgmasters(dot)net> wrote:
> You should produce a new version by then that addresses Alvaro's
> concerns and I imagine that will require quite a bit of discussion and
> work.
I'll get working.
I'll produce a patch with two alternate versions, one with and one without
the GIN operators.
On 3/7/18 5:43 AM, Alvaro Herrera wrote:
> so the performance was measured to see if the GIN operator was
> worth it, and the numbers are pretty confusing (the test don't seem
> terribly exhaustive; some numbers go up, some go down, it doesn't appear
> that the tests were run more than once for each case therefore the
> numbers are pretty noisy
>
Any ideas how to perform more exhaustive tests ?
On 3/26/18 4:50 PM, Mark Rofail wrote:
> > On Wed, Jan 31, 2018 at 1:52 AM, Andreas Karlsson
> <andreas(at)proxel(dot)se> wrote:
> >
> > The issue I see is that
> > ginqueryarrayextract() needs to make a copy of the search key but to
> do
> > so it needs to know the type of anyelement (to know if it needs to
> > detoast, etc). But there is as far as I can tell no way to check the
> > type of anyelement in this context.
> >
> > If there is any way to obtain a copy of the datum I would be more than
> > happy to integrate the GIN operator to the patch.
as I said before we need a way to obtain a copy of a datum to comply with
the context of ginqueryarrayextract()
Best Regards
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2018-04-10 13:57:35 | Re: crash with sql language partition support function |
Previous Message | Pavan Deolasee | 2018-04-10 13:54:17 | Re: Bugs in TOAST handling, OID assignment and redo recovery |