Re: BUG #1145: silent REVOKE failures

Fabien COELHO wrote:
>
> > >> For the TODO, I would suggest something general:
> > >> - fix grant/revoke wrt SQL standard, validate errors, warnings and successes.
> >
> > > Tom, is this done?
> >
> > I think so. At least, we're a lot closer to spec than we were.
>
> Indeed.
>
> Maybe the validation part could be improved somehow, with test cases
> build from the sql specification and the expected behavior checked
> against the actual behavior.
>
> What I derived from the many discussion and time I spent on the standard
> is that there is a lot of subtlety involved.
>
> So maybe the following TODO could be kept:
>
> - validate grant/revoke (error, warning, success0 wrt sql standard
>
> I may be interested in implementing ROLEs someday, and such tests would be
> welcome just to check that nothing is broken.

Unless someone can say it is wrong I am not inclinded to add a TODO item
that is only possible.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073