Re: [HACKERS] [BUGS] BUG #14682: row level security not work with partitioned table

On Wed, Jun 7, 2017 at 9:49 AM, Mike Palmiotto <mike.palmio...@crunchydata.com> wrote: > One thing that concerns me is the first EXPLAIN plan from regress_rls_dave: > +EXPLAIN (COSTS OFF) SELECT * FROM part_document WHERE f_leak(dtitle); > +

Re: [HACKERS] [BUGS] BUG #14682: row level security not work with partitioned table

the planner, or is it a bug? > > +SELECT * FROM part_document; > + did | cid | dlevel | dauthor | dtitle > +-+-++---+- > + 1 | 11 | 1 | regress_rls_bob | my first novel > Adding an "ORDER BY did" as well here

Re: [HACKERS] [BUGS] BUG #14682: row level security not work with partitioned table

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...@joeconw

Re: [HACKERS] [BUGS] BUG #14682: row level security not work with partitioned table

n 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. Thanks, -- Mike Palmiotto Software Engineer Crunchy Data Solut

Re: [HACKERS] partitioned tables and contrib/sepgsql

ssion tests before committing, but I expect we'll > have those by end of today and be able to commit the rest tomorrow. Attached are the regression test updates for partitioned tables. Thanks, -- Mike Palmiotto Software Engineer Crunchy Data Solutions https://crunchydata.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] partitioned tables and contrib/sepgsql

On Wed, Apr 5, 2017 at 1:22 PM, Mike Palmiotto <mike.palmio...@crunchydata.com> wrote: > On Wed, Apr 5, 2017 at 1:19 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: >> Peter Eisentraut <peter.eisentr...@2ndquadrant.com> writes: >>> On 4/5/17 12:04, Tom Lane wrote: >

Re: [HACKERS] partitioned tables and contrib/sepgsql

ery is explained. > > I stand by my previous patch suggestion, except that we can replace > the parenthetical comment with something like "(We don't care if > redefines "true"/"false"; those are close enough.)". > Sounds good. Updated patchset will incl

Re: [HACKERS] partitioned tables and contrib/sepgsql

; far away. :-) > > OBE -- I have scheduled time in 30 minutes from now, after I have gotten > my first cup of coffee ;-) After some discussion off-list, I've rebased and udpated the patches. Please see attached for further review. Thanks, -- Mike Palmiotto Software Engineer Crunchy Da

Re: [HACKERS] partitioned tables and contrib/sepgsql

On Fri, Mar 31, 2017 at 8:23 PM, Robert Haas <robertmh...@gmail.com> wrote: > On Fri, Mar 31, 2017 at 2:14 PM, Mike Palmiotto > <mike.palmio...@crunchydata.com> wrote: >> Attached you will find two patches, which were rebased on master as of >> 156d388 (applied wit

Re: [HACKERS] partitioned tables and contrib/sepgsql

On Fri, Mar 31, 2017 at 2:14 PM, Mike Palmiotto <mike.palmio...@crunchydata.com> wrote: > On Mon, Mar 27, 2017 at 12:09 PM, Mike Palmiotto > <mike.palmio...@crunchydata.com> wrote: >> On Mon, Mar 27, 2017 at 11:46 AM, Robert Haas <robertmh...@gmail.com> wrote: >

Re: [HACKERS] partitioned tables and contrib/sepgsql

On Mon, Mar 27, 2017 at 12:09 PM, Mike Palmiotto <mike.palmio...@crunchydata.com> wrote: > On Mon, Mar 27, 2017 at 11:46 AM, Robert Haas <robertmh...@gmail.com> wrote: >> >> Note that sepgsql hasn't been updated to work with RLS yet, either, >> but we didn't re

Re: [HACKERS] partitioned tables and contrib/sepgsql

On Mon, Mar 27, 2017 at 11:46 AM, Robert Haas <robertmh...@gmail.com> wrote: > On Mon, Mar 27, 2017 at 11:22 AM, Mike Palmiotto > <mike.palmio...@crunchydata.com> wrote: >> On Mon, Mar 27, 2017 at 10:47 AM, Robert Haas <robertmh...@gmail.com> wrote: >>>

Re: [HACKERS] partitioned tables and contrib/sepgsql

-alter hook in StoreCatalogInheritance1. It seems like it may just be an issue of adding the RELKIND_PARTITIONED_TABLE to sepgsql_relation_post_create. -- Mike Palmiotto Software Engineer Crunchy Data Solutions https://crunchydata.com -- Sent via pgsql-hackers mailing list (pgsql-

Re: [HACKERS] partitioned tables and contrib/sepgsql

On Thu, Mar 9, 2017 at 9:47 AM, Stephen Frost wrote: > Greetings, > > While going over the contrib modules, I noticed that sepgsql was not > updated for partitioned tables. What that appears to mean is that it's > not possible to define labels on partitioned tables. As I