From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | "Marc G(dot) Fournier" <scrappy(at)hub(dot)org> |
Cc: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Thomas Lockhart <lockhart(at)fourpalms(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql-server/src backend/tcop/postgres.cbacke |
Date: | 2002-08-13 13:43:56 |
Message-ID: | 25205.1029246236@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
"Marc G. Fournier" <scrappy(at)hub(dot)org> writes:
> One thought at the back of my mind is why not have something like a
> 'PG_VERSION' for XLOG? Maybe something so simple as a text file in both
> the data and xlog directory that just contains a timestamp from the
> initdb? then, when you startup postmaster with a -X option, it compares
> the two files and makes sure that they belong to each other?
While that isn't a bad idea, it seems to me that it's adding mechanism
to get around a problem that we don't need to have in the first place.
The only reason this risk exists is that the patch changes a monolithic
postmaster option (-D) into two independent options (-D/-X) that in
reality should never be independent.
Essentially, you're proposing Kevlar shoes as a solution for the problem
that you want to walk around carrying a loaded gun aimed at your foot.
The shoes might be a good idea anyway, but the primary problem is
elsewhere...
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Greg Copeland | 2002-08-13 13:48:45 | Re: [COMMITTERS] pgsql-server/src |
Previous Message | Tom Lane | 2002-08-13 13:24:20 | Re: [COMMITTERS] pgsql-server/src backend/tcop/postgres.cbacke |
From | Date | Subject | |
---|---|---|---|
Next Message | Don Baccus | 2002-08-13 13:48:28 | Re: OOP real life example (was Re: Why is MySQL more |
Previous Message | Zeugswetter Andreas SB SD | 2002-08-13 13:26:30 | Re: [GENERAL] Linux Largefile Support In Postgresql RPMS |