On Fri, Mar 24, 2023 at 2:14 PM Jelte Fennema <postgres@jeltef.nl> wrote:
> Yes, I totally agree. I now realise that wasn't clear at all from the wording in my previous email. I'm fine with
bothbehaviours. I mainly meant that if we actually think the new behaviour is better (which honestly I'm not convinced
ofyet), then some follow up patch would probably be good. I definitely don't want to block this patch on any of that
though.Both behaviors would be vastly better than the current one in my opinion. So if others wanted the behaviour in
yourpatch, I'm completely fine with that.
Makes sense. I hope a few more people will comment on what they think
we should do here, especially Andres and Noah.
> > Yeah, if we stick with the current approach we should probably add
> > tests for that stuff.
>
> Even if we don't, we should still have tests showing that the security restrictions that we intend to put in place
actuallydo their job.
Yeah, I just don't want to write the tests and then decide to change
the behavior and then have to write them over again. It's not so much
fun that I'm yearning to do it twice.
--
Robert Haas
EDB: http://www.enterprisedb.com