From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>, Andreas Seltenreich <seltenreich(at)gmx(dot)de>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Simon Riggs <simon(at)2ndquadrant(dot)com> |
Subject: | Re: [sqlsmith] Planner crash on foreign table join |
Date: | 2017-04-09 00:13:56 |
Message-ID: | 1952.1491696836@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> On Sat, Apr 8, 2017 at 3:57 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> I think it's pretty dubious to change this, honestly. Just because it
>> would have caught this one bug doesn't make it an especially valuable
>> thing in general. Bytes are still not free.
> What I think I might do is write a trial patch that turns Bitmapsets
> into Nodes, and see if it catches any other existing bugs. If it does
> not, that would be good evidence for your position.
I made the attached quick-hack patch, and found that check-world
passes just fine with it. That's not complete proof that we have
no other bugs of this ilk, but it definitely supports the idea
that we don't really need to add the overhead. I'll just put this
in the archives for possible future reference.
(Or perhaps Andreas would like to try bashing on a copy with this
installed.)
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
use-isa-checks-for-bitmapsets.patch | text/x-diff | 8.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2017-04-09 00:26:15 | Re: recent deadlock regression test failures |
Previous Message | Josh Berkus | 2017-04-08 21:31:52 | Re: 2017-03 CF Closed |