Thread: datistemplate of pg_database does not behave as per description in documentation

* Magnus Hagander (magnus@hagander.net) wrote:
> However, that also raises a third option. We could just drop the idea if
> datistemplate completely, and remove the column. Since clearly it's not
> actually doing anything, and people seem to have been happy with that for a
> while, why do we need it in the first place?

It's a bit of extra meta-data which can be nice to have (I know we use
that field for our own purposes..).  On the flip side, I've never liked
that you have to update pg_database to change it, so perhaps we should
get rid of it and remove that temptation.

The other field we regularly update in pg_database is datallowconn...
Would love to see that as an actual ALTER DATABASE command instead...
Thanks,
    Stephen

Magnus Hagander <magnus@hagander.net> writes:
> On Thu, Mar 27, 2014 at 9:12 AM, Rajeev rastogi
> <rajeev.rastogi@huawei.com>wrote:
>> But current code does not behave in this manner.  Even if dbistemplate of
>> database is false, still it allows to be used as template database.

> AFAICT, the *only* thing datistemplate is used is to set parameters in
> autovacuum.

Huh?  The code comment is perfectly clear:
   /*    * Permission check: to copy a DB that's not marked datistemplate, you    * must be superuser or the owner
thereof.   */   if (!src_istemplate)   {       if (!pg_database_ownercheck(src_dboid, GetUserId()))
ereport(ERROR,                  (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),                    errmsg("permission denied
tocopy database \"%s\"",                           dbtemplate)));   }
 

I agree we need to make the docs match the code, but changing behavior
that's been like that for ten or fifteen years isn't the answer.

(Changing code and failing to adjust the adjacent comment is even less
of an answer.)
        regards, tom lane

<p dir="ltr"><br /> On Mar 27, 2014 12:45 PM, "Tom Lane" <<a
href="mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>>wrote:<br /> ><br /> > Magnus Hagander <<a
href="mailto:magnus@hagander.net">magnus@hagander.net</a>>writes:<br /> > > On Thu, Mar 27, 2014 at 9:12 AM,
Rajeevrastogi<br /> > > <<a href="mailto:rajeev.rastogi@huawei.com">rajeev.rastogi@huawei.com</a>>wrote:<br
/>> >> But current code does not behave in this manner.  Even if dbistemplate of<br /> > >> database
isfalse, still it allows to be used as template database.<br /> ><br /> > > AFAICT, the *only* thing
datistemplateis used is to set parameters in<br /> > > autovacuum.<br /> ><br /> > Huh?  The code comment
isperfectly clear:<br /> ><br /> >     /*<br /> >      * Permission check: to copy a DB that's not marked
datistemplate,you<br /> >      * must be superuser or the owner thereof.<br /> >      */<br /> >     if
(!src_istemplate)<br/> >     {<br /> >         if (!pg_database_ownercheck(src_dboid, GetUserId()))<br /> >  
         ereport(ERROR,<br /> >                     (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),<br /> >            
        errmsg("permission denied to copy database \"%s\"",<br /> >                             dbtemplate)));<br />
>    }<br /> ><br /> > I agree we need to make the docs match the code, but changing behavior<br /> >
that'sbeen like that for ten or fifteen years isn't the answer.<p dir="ltr">Hah, I hadn't done more than git grep for
datistemplate:-) that's what I get for taking shortcuts... <p dir="ltr">But yes, fixing  the documentation is what I
advocateas well. <br /><p dir="ltr">> (Changing code and failing to adjust the adjacent comment is even less<br />
>of an answer.)<p dir="ltr">I didn't even look at the suggested patch.. <p dir="ltr">/Magnus  

On 27 March 2014 17:16, Tom Lane Wrote:

> I agree we need to make the docs match the code, but changing behavior
> that's been like that for ten or fifteen years isn't the answer.

Sounds good.

Please find the attached patch with only documentation change.

Thanks and Regards,
Kumar Rajeev Rastogi

From: "Rajeev rastogi" <rajeev.rastogi@huawei.com>
> Please find the attached patch with only documentation change.

I marked this as ready for committer.  The patch is good because it matches 
the description in the following page:

http://www.postgresql.org/docs/devel/static/manage-ag-templatedbs.html

Regards
MauMau

On Wed, Jun 18, 2014 at 7:47 PM, MauMau <maumau307@gmail.com> wrote:
> From: "Rajeev rastogi" <rajeev.rastogi@huawei.com>
>
>> Please find the attached patch with only documentation change.
>
>
> I marked this as ready for committer.  The patch is good because it matches
> the description in the following page:
>
> http://www.postgresql.org/docs/devel/static/manage-ag-templatedbs.html

ISTM that this patch has already been committed by Bruce. Please see
the commit 72590b3a69baaf24d1090a2c2ceb9181be34043e

Regards,

-- 
Fujii Masao

From: "Fujii Masao" <masao.fujii@gmail.com>
> On Wed, Jun 18, 2014 at 7:47 PM, MauMau <maumau307@gmail.com> wrote:
>> I marked this as ready for committer.  The patch is good because it 
>> matches
>> the description in the following page:
>>
>> http://www.postgresql.org/docs/devel/static/manage-ag-templatedbs.html
>
> ISTM that this patch has already been committed by Bruce. Please see
> the commit 72590b3a69baaf24d1090a2c2ceb9181be34043e

Oh, the devel doc certainly reflects the change:

http://www.postgresql.org/docs/devel/static/catalog-pg-database.html

I marked this as committed.

I hope Magnus-san's enhancements to the CommitFest app will enable the 
CommitFest entry to be automatically updated at git commit.

Regards
MauMau