| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Amit Langote <amitlangote09(at)gmail(dot)com> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, amul sul <sulamul(at)gmail(dot)com> |
| Subject: | Re: hyrax vs. RelationBuildPartitionDesc |
| Date: | 2019-06-07 12:51:56 |
| Message-ID: | CA+TgmoZxXR3k=3V2UsQN2EV3VQ455oFJK1+zAn8AzzbO+P9sPg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Jun 6, 2019 at 2:48 AM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> Attached is a patch that applies on top of Robert's pdoldcxt-v1.patch,
> which seems to fix this issue for me.
Yeah, that looks right. I think my patch was full of fuzzy thinking
and inadequate testing; thanks for checking it over and coming up with
the right solution.
Anyone else want to look/comment?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2019-06-07 12:55:36 | Re: Custom table AMs need to include heapam.h because of BulkInsertState |
| Previous Message | Robert Haas | 2019-06-07 12:43:21 | Re: tableam: abstracting relation sizing code |