Re: Add pg_file_sync() to adminpack
Julien Rouhaud <[email protected]> Tue, 14 Jan 2020 23:12:18 +0100
| Newsgroups | gmane.comp.db.postgresql.devel.general |
|---|---|
| Message-ID | <CAOBaU_bscJbeHMuMgu3z2HdT1f49z6cmXtbaBiLH+QyEwi-i+g@mail.gmail.com> |
--0000000000005fad67059c20e330 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Le mar. 14 janv. 2020 =C3=A0 22:57, Stephen Frost <[email protected]> a = =C3=A9crit : > Greetings, > > * Julien Rouhaud ([email protected]) wrote: > > On Fri, Jan 10, 2020 at 10:50 AM Fujii Masao <[email protected]> > wrote: > > > On Thu, Jan 9, 2020 at 10:39 PM Julien Rouhaud <[email protected]> > wrote: > > > > I think that pg_write_server_files should be allowed to call that > > > > function by default. > > > > > > But pg_write_server_files users are not allowed to execute > > > other functions like pg_file_write() by default. So doing that > > > change only for pg_file_sync() looks strange to me. > > > > Ah indeed. I'm wondering if that's an oversight of the original > > default role patch or voluntary. > > It was intentional. > ok, thanks for the clarification. > --0000000000005fad67059c20e330 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D= "gmail_attr">Le mar. 14 janv. 2020 =C3=A0 22:57, Stephen Frost <<a href= =3D"mailto:[email protected]">[email protected]</a>> a =C3=A9crit=C2= =A0:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;= border-left:1px #ccc solid;padding-left:1ex">Greetings,<br> <br> * Julien Rouhaud (<a href=3D"mailto:[email protected]" target=3D"_blank" r= el=3D"noreferrer">[email protected]</a>) wrote:<br> > On Fri, Jan 10, 2020 at 10:50 AM Fujii Masao <<a href=3D"mailto:mas= [email protected]" target=3D"_blank" rel=3D"noreferrer">masao.fujii@gmail.= com</a>> wrote:<br> > > On Thu, Jan 9, 2020 at 10:39 PM Julien Rouhaud <<a href=3D"mai= lto:[email protected]" target=3D"_blank" rel=3D"noreferrer">rjuju123@gmail= .com</a>> wrote:<br> > > > I think that pg_write_server_files should be allowed to call= that<br> > > > function by default.<br> > ><br> > > But pg_write_server_files users are not allowed to execute<br> > > other functions like pg_file_write() by default. So doing that<br= > > > change only for pg_file_sync() looks strange to me.<br> > <br> > Ah indeed.=C2=A0 I'm wondering if that's an oversight of the o= riginal<br> > default role patch or voluntary.<br> <br> It was intentional.<br></blockquote></div></div><div dir=3D"auto"><br></div= ><div dir=3D"auto">ok, thanks for the clarification.=C2=A0</div><div dir=3D= "auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style= =3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> </blockquote></div></div></div> --0000000000005fad67059c20e330--