From: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | a(dot)korotkov(at)postgrespro(dot)ru |
Cc: | sk(at)zsrv(dot)org, david(at)nlpgo(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Index-only scan returns incorrect results when using a composite GIST index with a gist_trgm_ops column. |
Date: | 2018-01-19 01:00:36 |
Message-ID: | 20180119.100036.129395697.horiguchi.kyotaro@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
Hello.
At Fri, 19 Jan 2018 01:16:56 +0300, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> wrote in <CAPpHfdvJeYrnAsXhKRfO_NMDUaWQyK+wyhcv4zOdRzTdfNCkkw(at)mail(dot)gmail(dot)com>
> On Thu, Jan 18, 2018 at 8:48 AM, Kyotaro HORIGUCHI <
> horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> wrote:
> > Index only scan is not usable in the case since the first index
> > column cannot be rechecked but check_index_only makes wrong
> > decision by the second occurance of "w'. There may be a chance
> > that recheck is not required but we cannot predict that until
> > actually acquire a tuple during execution.
> >
>
> I didn't get this point. Same column is indexed twice using two
> different opclasses. The first opclass doesn't support index-only scan,
> while the second opclass does support it. So, if the first opclass needs
> recheck then it can be rechecked using the value got from the second
> opclass. Thus, I think we can potentially support index-only scan
> in this case.
> Another thing is that it's probably hard to do in our current
> optimizer/executor/am
> infrastructure. And assuming that use-case is quite narrow, and it looks
To be exact, you're right. My description was abridged by
omitting exactly what you mentioned above. The complexity needed
to allow that doesn't seem worth adding.
> OK to just disable such feature as bug fix.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Kyotaro HORIGUCHI | 2018-01-19 01:54:53 | Re: [HACKERS] [BUGS] Bug in Physical Replication Slots (at least 9.5)? |
Previous Message | Alexander Korotkov | 2018-01-18 22:16:56 | Re: Index-only scan returns incorrect results when using a composite GIST index with a gist_trgm_ops column. |
From | Date | Subject | |
---|---|---|---|
Next Message | David Fetter | 2018-01-19 01:51:31 | Re: make_etags: avoid recursive symbolic creation. |
Previous Message | Michael Paquier | 2018-01-19 00:40:44 | Re: Add default role 'pg_access_server_files' |