From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pg_verify_checksums vs windows |
Date: | 2018-08-30 11:32:02 |
Message-ID: | CAA4eK1+NRkO9+fD5p=STTDKnubfKMPA4cnWf1zY141k-0bXDJQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Aug 29, 2018 at 5:17 PM Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>
> On Wed, Aug 29, 2018 at 1:44 PM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>>
>> On Wed, Aug 29, 2018 at 5:05 PM Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>> >
>> > On Wed, Aug 29, 2018 at 1:31 PM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>> >>
>> >> So, I think we need to open the file in binary mode as in other parts
>> >> of the code. The attached patch fixes the problem for me.
>> >>
>> >> Thoughts?
>> >
>> >
>> > Yikes. Yes, I believe you are correct, and that looks like the correct fix.
>> >
>> > I wonder why this was not caught on the buildfarm. We do have regression tests for it, AFAIK?
>> >
>>
>> I am not able to find regression tests for it, but maybe I am not
>> seeing it properly. By any chance, you have removed it during revert
>> of ""Allow on-line enabling and disabling of data checksums".
>>
>
> Oh meh. You are right, it's in the reverted patch, I was looking in the wrong branch :/ Sorry about that. And that certainly explains why we don't have it.
>
Okay. I will commit this in a day or so after once verifying it on
PG11 as well. I think this needs to be backpatched, let me know if
you think otherwise.
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2018-08-30 11:34:52 | Re: pg_verify_checksums vs windows |
Previous Message | Alexander Korotkov | 2018-08-30 11:28:20 | Re: Dimension limit in contrib/cube (dump/restore hazard?) |