On Wed, Mar 15, 2017 at 1:13 AM, Robert Haas wrote:
> On Mon, Mar 13, 2017 at 8:51 PM, Michael Paquier
> wrote:
>>> I agree with that, but I propose the attached version instead. It
>>> seems cleaner to have the entire test for setting BM_PERMANENT in one
>>> place rather than splitting it up as
On Mon, Mar 13, 2017 at 8:51 PM, Michael Paquier
wrote:
>> I agree with that, but I propose the attached version instead. It
>> seems cleaner to have the entire test for setting BM_PERMANENT in one
>> place rather than splitting it up as you did.
>
> Fine for me. You may want to update the commen
On Tue, Mar 14, 2017 at 4:46 AM, Robert Haas wrote:
> On Wed, Jan 25, 2017 at 7:14 PM, Michael Paquier
> wrote:
>> (Adding Robert in CC.)
>>
>> On Thu, Jan 26, 2017 at 4:34 AM, Wang Hao wrote:
>>> An unlogged table has an initialization fork. The initialization fork does
>>> not have an BM_PERMA
On Wed, Jan 25, 2017 at 7:14 PM, Michael Paquier
wrote:
> (Adding Robert in CC.)
>
> On Thu, Jan 26, 2017 at 4:34 AM, Wang Hao wrote:
>> An unlogged table has an initialization fork. The initialization fork does
>> not have an BM_PERMANENT flag when get a buffer.
>> In checkpoint (not shutdown or
On Sat, Mar 11, 2017 at 12:03 AM, Artur Zakirov
wrote:
> Because BM_PERMANENT is used for init forks of unlogged indexes now.
Yes, indeed.
--
Michael
diff --git a/contrib/bloom/blinsert.c b/contrib/bloom/blinsert.c
index 913f1f8a51..3557b106d8 100644
--- a/contrib/bloom/blinsert.c
+++ b/contrib/
On 10.03.2017 04:00, Michael Paquier wrote:
On Thu, Mar 9, 2017 at 10:25 PM, Artur Zakirov wrote:
I think this is good fixes. I've checked them. And in my opinion they are
correct.
The code also is good.
Having something with conflicts is not nice, so attached is a rebased version.
Thank y
On Thu, Mar 9, 2017 at 10:25 PM, Artur Zakirov wrote:
> I think this is good fixes. I've checked them. And in my opinion they are
> correct.
>
> The code also is good.
Having something with conflicts is not nice, so attached is a rebased version.
> I have run regression and TAP tests. They all p
Hello,
I wanted to review the patch. But the patch is applied with errors. I've
rebased the local copy and have done review on it. I'm not sure is it
properly to send rebased patch by reviewer, so I haven't sent it to
avoid confuses.
On 29.01.2017 17:00, Michael Paquier wrote:
Attached is w
On Thu, Jan 26, 2017 at 9:14 AM, Michael Paquier
wrote:
> So the patch attached fixes the problem by changing BufferAlloc() in
> such a way that initialization forks are permanently written to disk,
> which is what you are suggesting. As a simple fix for back-branches
> that's enough, though on HE
(Adding Robert in CC.)
On Thu, Jan 26, 2017 at 4:34 AM, Wang Hao wrote:
> An unlogged table has an initialization fork. The initialization fork does
> not have an BM_PERMANENT flag when get a buffer.
> In checkpoint (not shutdown or end of recovery), it will not write to disk.
> after a crash rec
An unlogged table has an initialization fork. The initialization fork does
not have an BM_PERMANENT flag when get a buffer.
In checkpoint (not shutdown or end of recovery), it will not write to disk.
after a crash recovery, the page of initialization fork will not correctly,
then make the main fork
11 matches
Mail list logo