On 06/06/2017 02:44 PM, Mike Palmiotto wrote: > On Tue, Jun 6, 2017 at 4:07 PM, Joe Conway <m...@joeconway.com> wrote: >> On 06/06/2017 11:57 AM, Mike Palmiotto wrote: >>> On Mon, Jun 5, 2017 at 10:36 AM, Robert Haas <robertmh...@gmail.com> wrote: >>>> On Mon, Jun 5, 2017 at 10:20 AM, Joe Conway <m...@joeconway.com> wrote: >>>>> Unless Robert objects, I'll work with Mike to get a fix posted and >>>>> committed in the next day or two. >>>> >>>> That would be great. Thanks. >>> >>> I have the updated patch with rowsecurity regression tests and rebased >>> on master. I've run these and verified locally by feeding >>> rowsecurity.sql to psql, but have yet to get the full regression suite >>> passing -- it's failing on the constraints regtest and then gets stuck >>> in recovery. Undoubtedly something to do with my >>> configuration/environment over here. I'm working through those issues >>> right now. In the meantime, if you want to see the regression tests as >>> they stand, please see the attached patch. >> >> The constraints test passes here, so presumably something you borked >> locally. I only see a rowsecurity failure, which is not surprising since >> your patch does not include the changes to expected output ;-) >> Please resend with src/test/regress/expected/rowsecurity.out included. > > It was indeed an issue on my end. Attached are the rowsecurity > regression tests and the expected out. Unsurprisingly, all tests pass, > because I said so. :) > > Let me know if you want me to make any revisions.
Thanks Mike. I'll take a close look to verify output correctnes, but I am concerned that the new tests are unnecessarily complex. Any other opinions on that? Joe -- Crunchy Data - http://crunchydata.com PostgreSQL Support for Secure Enterprises Consulting, Training, & Open Source Development
signature.asc
Description: OpenPGP digital signature