| From: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> |
|---|---|
| To: | Anastasia Lubennikova <lubennikovaav(at)gmail(dot)com>, Thom Brown <thom(at)linux(dot)com> |
| Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Index-only scans for GiST. |
| Date: | 2015-02-12 16:40:40 |
| Message-ID: | 54DCD788.1080003@vmware.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 02/12/2015 12:40 PM, Anastasia Lubennikova wrote:
> Thanks for answer.
> Now it seems to be applied correctly.
Thanks, it would be great to get this completed.
This still leaks memory with the same test query as earlier. The leak
seems to be into a different memory context now; it used to be to the
"GiST scan context", but now it's to the ExecutorState context. See
attached patch which makes the leak evident.
Looking at my previous comments from August:
* What's the reason for turning GISTScanOpaqueData.pageData from an
array to a List?
* Documentation is missing.
- Heikki
| Attachment | Content-Type | Size |
|---|---|---|
| show-gistgettuple-leak-2.patch | application/x-patch | 520 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2015-02-12 16:44:41 | Re: assessing parallel-safety |
| Previous Message | Robert Haas | 2015-02-12 16:37:32 | Re: assessing parallel-safety |