Re: BUG #10675: alter database set tablespace and unlogged table

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>
Cc: Andres Freund <andres(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "maxim(dot)boguk" <maxim(dot)boguk(at)gmail(dot)com>, Pg Bugs <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: BUG #10675: alter database set tablespace and unlogged table
Date: 2014-08-08 12:44:48
Message-ID: CAHGQGwEJt7HDAAD2y4KD=iXYmtNpxLXKUgaAekGXcL9GVbCGgQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Fri, Aug 8, 2014 at 4:08 PM, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com> wrote:
> On Thu, Aug 7, 2014 at 5:54 PM, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>>
>>
>>
>> What's the status of this?
>
>
> There was no clear consensus. The thread has patches per both the
> approaches. I tried to bring it up couple of times, last time just before
> the minor releases were out. But did not get any attention.

I vote to make checkpoint flush unlogged tables only when the special flag
(that maybe we will introduce) is set. Unlogged tables are not flushed by
manual checkpoint. Per Robert, that's originally intentional. No dangerous
risk that may cause has been reported yet. So I don't think that we need to
change manual checkpoint so that it always flushes unlogged tables, at least
for now.

OTOH, I agree to extend manual checkpoint so that it can flush unlogged
table to shorten the time of subsequent shutdown checkpoint. For example,
we can implement something like "CHECKPOINT [ALL | PERMANENT | UNLOGGED]".
But that's a feature request. We should work on this later, not for now.

>> ISTM that the patch has not been applied yet. Right?
>>
>
> Right. I've now added it to the commitfest so that we don't lose track of
> it.

Thanks!

Regards,

--
Fujii Masao

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message pjduplooy 2014-08-08 14:28:16 Re: BUG #11039: installation fails when trying to install C++ redistributable
Previous Message mozsar.laszlo 2014-08-08 12:22:49 BUG #11130: Case condition error