Amit Kapila <amit.kapila16@gmail.com> writes:
> On Fri, Jan 24, 2014 at 9:10 AM, Amit Kapila <amit.kapila16@gmail.com> wrote:
>> On Fri, Jan 24, 2014 at 2:38 AM, Robert Haas <robertmhaas@gmail.com> wrote:
>>> I mean, if
>>> there's a GUC that controls the event source name, then it can be
>>> changed between restarts, regardless of what you call it.
>> Yes, but not default values (when user don't provide any value
>> for event_soource). Here the question is about default value of
>> event_source.
> To proceed with the review of this patch, I need to know about
> whether appending version number or any other constant togli
> Default Event Source name is acceptable or not, else for now
> we can remove this part of code from patch and handle non-default
> case where the change will be that pg_ctl will enquire non-default
> event_source value from server.
> Could you please let me know your views about same?
Unless I'm missing something, this entire thread is a tempest in a teapot,
because the default event_source value does not matter, because *by
default we don't log to eventlog*. The presumption is that if the user
turns on logging to eventlog, it's his responsibility to first make sure
that event_source is set to something appropriate. And who's to say that
plain "PostgreSQL" isn't what he wanted, anyway? Even if he's got
multiple servers on one machine, maybe directing all their logs to the
same place is okay by him.
Also, those who don't run multiple servers are probably not going to
thank us for moving their logs around unnecessarily.
In short, I think we should just reject this idea as introducing more
problems than it solves, and not fully solving even the problem it
purports to solve.
Possibly there's room for a documentation patch reminding users to
make sure that event_source is set appropriately before they turn
on eventlog.
regards, tom lane