2017-02-05 7:54 GMT+01:00 Pavan Deolasee <pavan.deola...@gmail.com>:

> On Sat, Feb 4, 2017 at 11:54 PM, Tom Lane <t...@sss.pgh.pa.us> wrote:
>> Based on Pavan's comments, I think trying to force this into next week's
>> releases would be extremely unwise.  If the bug went undetected this long,
>> it can wait for a fix for another three months.
> Yes, I think bug existed ever since and went unnoticed. One reason could
> be that the race happens only when the new index turns HOT updates into
> non-HOT updates. Another reason could be that we don't have checks in place
> to catch these kinds of corruption. Having said that, since we have
> discovered the bug, at least many 2ndQuadrant customers have expressed
> worry and want to know if the fix will be available in 9.6.2 and other
> minor releases.  Since the bug can lead to data corruption, the worry is
> justified. Until we fix the bug, there will be a constant worry about using
> CIC.
> If we can have some kind of band-aid fix to plug in the hole, that might
> be enough as well. I tested my first patch (which will need some polishing)
> and that works well AFAIK. I was worried about prepared queries and all,
> but that seems ok too. RelationGetIndexList() always get called within
> ExecInitModifyTable. The fix seems quite unlikely to cause any other side
> effects.
> Another possible band-aid is to throw another relcache invalidation in
> CIC. Like adding a dummy index_set_state_flags() within yet another
> transaction. Seems ugly, but should fix the problem for now and not cause
> any impact on relcache mechanism whatsoever.
> That seems better than
>> risking new breakage when it's barely 48 hours to the release wrap
>> deadline.  We do not have time to recover from any mistakes.
> I'm not sure what the project policies are, but can we consider delaying
> the release by a week for issues like these? Or do you think it will be
> hard to come up with a proper fix for the issue and it will need some
> serious refactoring?

I agree with Pavan - a release with known important bug is not good idea.


> Thanks,
> Pavan
> --
>  Pavan Deolasee                   http://www.2ndQuadrant.com/
>  PostgreSQL Development, 24x7 Support, Training & Services

Reply via email to