Bruce Momjian <[EMAIL PROTECTED]> writes:
> The bottom line is that I have no problem with making this change, but I
> want to be sure it is a full solution we aren't going to have to revisit
> later. I don't want to make the change and then find out we rushed it
> and have to make more adjustment
The bottom line is that I have no problem with making this change, but I
want to be sure it is a full solution we aren't going to have to revisit
later. I don't want to make the change and then find out we rushed it
and have to make more adjustments in later releases.
---
Bruce Momjian <[EMAIL PROTECTED]> writes:
> Tom Lane wrote:
>> started because setting these variables via ALTER USER fails to work.
> It fails to work if logging was already on and someone wants to turn it
> off via ALTER USER, and that matches the expected behavior.
Not if a superuser does it;
Bruce Momjian <[EMAIL PROTECTED]> writes:
> I would be glad to see USERLIMIT gone, even though I wrote it. However,
> I feel we are removing user's ability of non-super users to turn logging
> on and off easily without really having thought through a mechanism to
> give them that.
I think that is
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > Tom Lane wrote:
> >> started because setting these variables via ALTER USER fails to work.
>
> > It fails to work if logging was already on and someone wants to turn it
> > off via ALTER USER, and that matches the expected behavior.
>
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > I would be glad to see USERLIMIT gone, even though I wrote it. However,
> > I feel we are removing user's ability of non-super users to turn logging
> > on and off easily without really having thought through a mechanism to
> > give t
Tom Lane wrote:
> The attached patch removes GUC's USERLIMIT variable category, changing
> all the affected variables to be plain SUSET, as per recent discussion.
>
> I also modified postgres.c so that variable settings coming from the
> client connection request packet (eg, from PGOPTIONS on the