Re: WITH CHECK and Column-Level Privileges

Dean,

* Dean Rasheed (dean(dot)a(dot)rasheed(at)gmail(dot)com) wrote:
> On 27 January 2015 at 22:45, Stephen Frost <sfrost(at)snowman(dot)net> wrote:
> > Here's the latest set with a few additional improvements (mostly
> > comments but also a couple missed #include's and eliminating unnecessary
> > whitespace changes). Unless there are issues with my testing tonight or
> > concerns raised, I'll push these tomorrow.
>
> I spotted a couple of minor things reading the patches:
>
> - There's a typo in the comment for the GetModifiedColumns() macros
> ("...stick in into...").

Fixed.

> - The new regression test is not tidying up properly after itself,
> because it's trying to drop the table t1 as the wrong user.

Urgh. Not sure how I managed to miss that; guess I was just too focused
on what I was testing. :)

> Other than that, this looks reasonable to me, and I think that for
> most common situations it won't reduce the detail in errors.

Thanks! I'll be pushing this soon (finally!).

Thanks again,

Stephen