Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-16 Thread Jan Wieck
Fabien COELHO wrote: OK, patch reverted at request of Tom and Peter. Please propose a more generalitzed soluion. Thanks. Sigh. You refuse a 10 lines patch to access a stupid opaque type in the backend. You both refuse some things to be done in the back end, and also to add what is needed to do it

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-16 Thread Fabien COELHO
I agree with you that work is needed in the acl area. > I hope you understand the larger scale of problems that would be created > if your patch got accepted. I would be ready to bet that I would be the only user for these functions. So large looks like me alone;-) Have a nice day, -- Fabien

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-04 Thread Fabien COELHO
> OK, patch reverted at request of Tom and Peter. Please propose a more > generalitzed soluion. Thanks. Sigh. You refuse a 10 lines patch to access a stupid opaque type in the backend. You both refuse some things to be done in the back end, and also to add what is needed to do it in userland.

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-04 Thread Fabien COELHO
Dear Tom, > http://developer.postgresql.org/docs/postgres/functions-misc.html > Arguably these functions do not belong right there, but that's hardly a > reason to think that they do not need documentation. Sure. I was planing to add something anyway. > Personally, though, I think that Peter's

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-04 Thread Fabien COELHO
> > Please find a attached a small patch that adds accessor functions > > for "aclitem" so that it is not an opaque datatype. > > Shouldn't this patch include some documentation updates? Well I thought about that, but I wasn't sure were to put it. There is no documentation at all about th

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-02 Thread Bruce Momjian
OK, patch reverted at request of Tom and Peter. Please propose a more generalitzed soluion. Thanks. --- Fabien COELHO wrote: > > Dear Tom, > > > http://developer.postgresql.org/docs/postgres/functions-misc.html > > Argua

[COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-02 Thread Bruce Momjian
CVSROOT:/cvsroot Module name:pgsql-server Changes by: [EMAIL PROTECTED] 04/05/02 10:38:28 Modified files: src/backend/utils/adt: acl.c src/include/catalog: catversion.h pg_proc.h src/include/utils: acl.h src/test/regress/expected: privileges

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-05-02 Thread Peter Eisentraut
Tom Lane wrote: > Ergo, my recommendation is to revert this change altogether. Fabien > should figure out the high-level description of what he wants to know > (at a level similar to has_table_privilege() and its ilk) and propose > server-side functions to implement that. I agree with that. ---

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-04-28 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > Neil Conway wrote: >> Shouldn't this patch include some documentation updates? > I talked to the author about this. Turns out none of our permission > functions have docs. Really? What is Table 9.37 in http://developer.postgresql.org/docs/postgres/fun

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-04-28 Thread Bruce Momjian
Neil Conway wrote: > On 26-Apr-04, at 11:06 AM, Bruce Momjian wrote: > > Please find a attached a small patch that adds accessor functions > > for "aclitem" so that it is not an opaque datatype. > > Shouldn't this patch include some documentation updates? I talked to the author about this

Re: [COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-04-27 Thread Neil Conway
On 26-Apr-04, at 11:06 AM, Bruce Momjian wrote: Please find a attached a small patch that adds accessor functions for "aclitem" so that it is not an opaque datatype. Shouldn't this patch include some documentation updates? -Neil ---(end of broadcast)-

[COMMITTERS] pgsql-server/src backend/utils/adt/acl.c inclu ...

2004-04-26 Thread Bruce Momjian
CVSROOT:/cvsroot Module name:pgsql-server Changes by: [EMAIL PROTECTED] 04/04/26 12:06:49 Modified files: src/backend/utils/adt: acl.c src/include/catalog: catversion.h pg_proc.h src/include/utils: acl.h src/test/regress/expected: privileges