This is a discussion on Re: Proposed Patchs within the pgsql Interfaces Pgadmin Support forums, part of the PostgreSQL category; --> > -----Original Message----- > From: Thomas Sondag [mailto:thomas.sondag@gmail.com] > Sent: 30 May 2006 13:42 > To: Dave Page > ...
| |||||||
| FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read |
| ||||
| > -----Original Message----- > From: Thomas Sondag [mailto:thomas.sondag@gmail.com] > Sent: 30 May 2006 13:42 > To: Dave Page > Cc: pgadmin-support@postgresql.org > Subject: Re: [pgadmin-support] Proposed Patchs > > I'm not sure this is the better way, because most of the time objects > are owned by the current user, so showing only group by default can be > a bit confusing. There's no reason we cannot use groups + current user. I do think it would be more confusing to have the behaviour work in precisely the opposite way to the privileges. > If you want to see the result with the "Show group ..." option try > the folowing patch . the main idea for AddRoles function was to remove > one sql query. > > I test this patch only with postgresql 8.1 and it stop > segfaulting now Think you forgot the patch :-( /D ---------------------------(end of broadcast)--------------------------- TIP 4: Have you searched our list archives? http://archives.postgresql.org |
| ||||
| 2006/5/30, Dave Page <dpage@vale-housing.co.uk>: > > > > -----Original Message----- > > From: Thomas Sondag [mailto:thomas.sondag@gmail.com] > > Sent: 30 May 2006 13:42 > > To: Dave Page > > Cc: pgadmin-support@postgresql.org > > Subject: Re: [pgadmin-support] Proposed Patchs > > > > I'm not sure this is the better way, because most of the time objects > > are owned by the current user, so showing only group by default can be > > a bit confusing. > > There's no reason we cannot use groups + current user. I do think it > would be more confusing to have the behaviour work in precisely the > opposite way to the privileges. > I'm agree, I wasn't see things that way, but that's different to the current behaviour. > > If you want to see the result with the "Show group ..." option try > > the folowing patch . the main idea for AddRoles function was to remove > > one sql query. > > > > I test this patch only with postgresql 8.1 and it stop > > segfaulting now > > Think you forgot the patch :-( > I hope you'll get it this time. Thomas ---------------------------(end of broadcast)--------------------------- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq |