On Tue, Nov 25, 2008 at 4:03 PM, Stephen Frost <[EMAIL PROTECTED]> wrote:
> * Alvaro Herrera ([EMAIL PROTECTED]) wrote:
>> I had a look at aclchk.c and didn't like your change to
>> objectNamesToOids; seems rather baroque. I changed it per the attached
>> patch.
>
> I've incorporated this change.
Alvaro,
* Alvaro Herrera ([EMAIL PROTECTED]) wrote:
> I had a look at aclchk.c and didn't like your change to
> objectNamesToOids; seems rather baroque. I changed it per the attached
> patch.
I've incorporated this change.
> Moreover I didn't very much like the way aclcheck_error_col is dealing
Alvaro Herrera wrote:
> I didn't check the rest of the code, so don't count this as a review.
I had a look at aclchk.c and didn't like your change to
objectNamesToOids; seems rather baroque. I changed it per the attached
patch.
Moreover I didn't very much like the way aclcheck_error_col is deal
Alvaro Herrera wrote:
> Alvaro Herrera wrote:
>
> > I'll probably fix both things and submit an updated version tomorrow.
>
> Here it is.
Really attached this time.
--
Alvaro Herrerahttp://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
co
Alvaro Herrera wrote:
> I'll probably fix both things and submit an updated version tomorrow.
Here it is. This applies cleanly to current CVS HEAD and passes the
regression tests. Apart from fixing the conflicts, I updated psql's \z
with the new array aggregate, and changed the Schema_pg_* decl
Stephen Frost wrote:
> Markus,
>
> * Markus Wanner ([EMAIL PROTECTED]) wrote:
> > Stephen Frost wrote:
> > > Attached patch has this fixed and still passes all regression tests,
> > > etc.
> >
> > Do you have an up-to-date patch laying around? The current one conflicts
> > with some CVS tip chang
Markus,
* Markus Wanner ([EMAIL PROTECTED]) wrote:
> Stephen Frost wrote:
> > Attached patch has this fixed and still passes all regression tests,
> > etc.
>
> Do you have an up-to-date patch laying around? The current one conflicts
> with some CVS tip changes.
No, not yet. I suspect the array_
Hello Stephen,
Stephen Frost wrote:
> Attached patch has this fixed and still passes all regression tests,
> etc.
Do you have an up-to-date patch laying around? The current one conflicts
with some CVS tip changes.
I didn't get around writing some docu, yet. Sorry.
Regards
Markus Wanner
--
S
Hello Stephen,
Stephen Frost wrote:
> This has been fixed in the attached patch.
Cool, thanks.
> If you could work on the documentation, that'd be great!
I'll give it a try.
Regards
Markus Wanner
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your
* Tom Lane ([EMAIL PROTECTED]) wrote:
> Stephen Frost <[EMAIL PROTECTED]> writes:
> > ... A case I just realized might be an issue is
> > doing a 'select 1 from x;' where you have *no* rights on x, or any
> > columns in it, would still get you the rowcount.
>
> Well, if you have table-level select
* Tom Lane ([EMAIL PROTECTED]) wrote:
> Stephen Frost <[EMAIL PROTECTED]> writes:
> > ... A case I just realized might be an issue is
> > doing a 'select 1 from x;' where you have *no* rights on x, or any
> > columns in it, would still get you the rowcount.
>
> Well, if you have table-level select
Stephen Frost <[EMAIL PROTECTED]> writes:
> ... A case I just realized might be an issue is
> doing a 'select 1 from x;' where you have *no* rights on x, or any
> columns in it, would still get you the rowcount.
Well, if you have table-level select on x, I would expect that to work,
even if your p
Markus, et al,
* Stephen Frost ([EMAIL PROTECTED]) wrote:
> I also wonder if you could use joins or something
> to extract information about columns you're not supposed to have access
> to, or where clauses, etc..
welp, I've done some additional testing and there's good news and bad, I
suppose.
Markus,
* Markus Wanner ([EMAIL PROTECTED]) wrote:
> Sorry, this took way longer than planned.
Beleive me, I understand. :)
> testdb=# GRANT TRUNCATE (single_col) ON test TO malory;
> GRANT
This has been fixed in the attached patch.
> Some privilege regression tests currently fail with your pa
Hi,
Markus Wanner wrote:
As mentioned in above, regression tests, documentation updates,
dependency handling, and actually implementing the permission checks all
remain. What I'm looking for feedback on are the changes to the
grammer, parser, catalog changes, psql output, etc.
Aha, good. So I
Hi,
Stephen Frost wrote:
I would suggest you review the updated patch (linked off the wiki page)
here:
http://archives.postgresql.org/message-id/[EMAIL PROTECTED]
That's the patch I've been talking about: file
colprivs_wip.20080902.diff.gz, dated Sept, 2nd.
It includes my comments about wh
Hi Markus,
* Markus Wanner ([EMAIL PROTECTED]) wrote:
> Stephen Frost wrote:
>> Comments welcome, apologies for it not being ready by 9/1. I'm
>> planning to continue working on it tomorrow, and throughout September
>> as opportunity allows (ie: when Josh isn't keeping me busy).
>
> I'm try
Hello Stephen,
Stephen Frost wrote:
Comments welcome, apologies for it not being ready by 9/1. I'm
planning to continue working on it tomorrow, and throughout September
as opportunity allows (ie: when Josh isn't keeping me busy).
I'm trying to review this patch. I could at least compile
Greetings,
* Stephen Frost ([EMAIL PROTECTED]) wrote:
> Comments welcome, apologies for it not being ready by 9/1. I'm
> planning to continue working on it tomorrow, and throughout September
> as opportunity allows (ie: when Josh isn't keeping me busy).
Here is an updated patch. I've adde
19 matches
Mail list logo
