Re: [HACKERS] Should we remove "not fast" promotion at all?

2014-05-06 Thread Simon Riggs
On 19 August 2013 09:20, Heikki Linnakangas wrote: > On 08.08.2013 20:15, Josh Berkus wrote: >> >> Bruce, all: >> >>> We seem to be all over the map with the fast promotion code --- some >>> people don't trust it, some people want an option to enable the old >>> method, and some people want the ol

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Heikki Linnakangas
On 19.08.2013 20:27, Tom Lane wrote: Bruce Momjian writes: On Mon, Aug 19, 2013 at 11:20:42AM +0300, Heikki Linnakangas wrote: I think "promote" file should trigger the fast promotion, and the filename to trigger the slow mode should be called "fallback_promote" or "safe_promote" or something

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Bruce Momjian
On Mon, Aug 19, 2013 at 01:27:29PM -0400, Tom Lane wrote: > Bruce Momjian writes: > > On Mon, Aug 19, 2013 at 11:20:42AM +0300, Heikki Linnakangas wrote: > >> I think "promote" file should trigger the fast promotion, and the > >> filename to trigger the slow mode should be called > >> "fallback_pr

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Tom Lane
Bruce Momjian writes: > On Mon, Aug 19, 2013 at 11:20:42AM +0300, Heikki Linnakangas wrote: >> I think "promote" file should trigger the fast promotion, and the >> filename to trigger the slow mode should be called >> "fallback_promote" or "safe_promote" or something like that. There >> wasn't any

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Bruce Momjian
On Mon, Aug 19, 2013 at 11:20:42AM +0300, Heikki Linnakangas wrote: > And it's even worse if you use 9.3 pg_ctl against a 9.2 server: it > will create a filed called "fast_promote" and return success, but it > won't actually do anything. > > I think "promote" file should trigger the fast promotion

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Kevin Grittner
Robert Haas wrote: > Heikki Linnakangas wrote: >> I think "promote" file should trigger the fast promotion, and >> the filename to trigger the slow mode should be called >> "fallback_promote" or "safe_promote" or something like that. >> There wasn't any good reason to change the filename primari

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Robert Haas
On Mon, Aug 19, 2013 at 4:20 AM, Heikki Linnakangas wrote: > Well, I don't see much harm in keeping the old behavior as an undocumented > escape hatch, as it is now. The way I'd phrase the current situation is > this: 9.3 now always does "fast promotion". However, for debugging and > testing purpo

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-19 Thread Heikki Linnakangas
On 08.08.2013 20:15, Josh Berkus wrote: Bruce, all: We seem to be all over the map with the fast promotion code --- some people don't trust it, some people want an option to enable the old method, and some people want the old method removed. Having read over this thread, the only reason given

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Tomonari Katsumata
Hi, I understood it's too late to change the feature. I hope it will be revised in 9.4! (2013/08/09 4:13), Josh Berkus wrote: > On 08/08/2013 11:01 AM, Andres Freund wrote: > >> I don't think anybody working on related areas of the code thinks it's >> rock solid. >> But anyway, I just don't see

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Josh Berkus
On 08/08/2013 11:01 AM, Andres Freund wrote: > I don't think anybody working on related areas of the code thinks it's > rock solid. > But anyway, I just don't see the downside of allowing problem > analysis. You're free to do more testing, review, whatever before the > release. I'm 100% with you

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Andres Freund
On 2013-08-08 10:51:45 -0700, Josh Berkus wrote: > On 08/08/2013 10:34 AM, Andres Freund wrote: > > On 2013-08-08 10:15:14 -0700, Josh Berkus wrote: > >> Either we have confidence is fast promotion, or we don't. If we don't > >> have confidence, then either (a) more testing is needed, or (b) it >

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Josh Berkus
On 08/08/2013 10:34 AM, Andres Freund wrote: > On 2013-08-08 10:15:14 -0700, Josh Berkus wrote: >> Either we have confidence is fast promotion, or we don't. If we don't >> have confidence, then either (a) more testing is needed, or (b) it >> shouldn't be the default. Again, here, we are coming up

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Andres Freund
On 2013-08-08 10:15:14 -0700, Josh Berkus wrote: > Bruce, all: > > > We seem to be all over the map with the fast promotion code --- some > > people don't trust it, some people want an option to enable the old > > method, and some people want the old method removed. > > Having read over this thre

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Josh Berkus
Bruce, all: > We seem to be all over the map with the fast promotion code --- some > people don't trust it, some people want an option to enable the old > method, and some people want the old method removed. Having read over this thread, the only reason given for retaining any ability to use "old

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Andres Freund
On 2013-08-08 12:50:31 -0400, Bruce Momjian wrote: > On Thu, Aug 8, 2013 at 01:27:35PM +0900, Michael Paquier wrote: > > > Why are we suddenly trying to make this even more complicated? It's too > > > late to redesign stuff without very good evidence that it's > > > needed. Renaming trigger files

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-08 Thread Bruce Momjian
On Thu, Aug 8, 2013 at 01:27:35PM +0900, Michael Paquier wrote: > > Why are we suddenly trying to make this even more complicated? It's too > > late to redesign stuff without very good evidence that it's > > needed. Renaming trigger files and changing their format certainly > > doesn't seem approp

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-07 Thread Michael Paquier
On Thu, Aug 8, 2013 at 12:24 PM, Andres Freund wrote: > On 2013-08-08 06:40:00 +0900, Michael Paquier wrote: >> On Tue, Aug 6, 2013 at 8:05 PM, Tomonari Katsumata >> wrote: >> > Hi, >> > >> > 2013/8/6 Tom Lane >> >> >> >> Fujii Masao writes: >> >> > On Tue, Aug 6, 2013 at 11:40 AM, Andres Freun

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-07 Thread Andres Freund
On 2013-08-08 06:40:00 +0900, Michael Paquier wrote: > On Tue, Aug 6, 2013 at 8:05 PM, Tomonari Katsumata > wrote: > > Hi, > > > > 2013/8/6 Tom Lane > >> > >> Fujii Masao writes: > >> > On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund > >> > wrote: > >> >> FWIW I'd rather keep plain promotion for

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-07 Thread Michael Paquier
On Tue, Aug 6, 2013 at 8:05 PM, Tomonari Katsumata wrote: > Hi, > > 2013/8/6 Tom Lane >> >> Fujii Masao writes: >> > On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund >> > wrote: >> >> FWIW I'd rather keep plain promotion for a release or two. TBH, I have >> >> a >> >> bit of trust issues regardin

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-07 Thread Andres Freund
On 2013-08-07 22:26:53 +0900, Fujii Masao wrote: > On Tue, Aug 6, 2013 at 1:07 PM, Tom Lane wrote: > > Fujii Masao writes: > >> On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund > >> wrote: > >>> FWIW I'd rather keep plain promotion for a release or two. TBH, I have a > >>> bit of trust issues reg

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-07 Thread Fujii Masao
On Tue, Aug 6, 2013 at 1:07 PM, Tom Lane wrote: > Fujii Masao writes: >> On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund >> wrote: >>> FWIW I'd rather keep plain promotion for a release or two. TBH, I have a >>> bit of trust issues regarding the new method, and I'd like to be able to >>> test po

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-06 Thread Tomonari Katsumata
Hi, 2013/8/6 Tom Lane > Fujii Masao writes: > > On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund > wrote: > >> FWIW I'd rather keep plain promotion for a release or two. TBH, I have a > >> bit of trust issues regarding the new method, and I'd like to be able to > >> test potential issues against

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Michael Paquier
On Tue, Aug 6, 2013 at 12:52 PM, Andres Freund wrote: > > > Fujii Masao schrieb: >>On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund >>wrote: >>> Hi, >>> >>> On 2013-08-06 03:24:58 +0900, Fujii Masao wrote: Hi all, We discussed the $SUBJECT in the following threads: >>http://www

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Tom Lane
Fujii Masao writes: > On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund wrote: >> FWIW I'd rather keep plain promotion for a release or two. TBH, I have a >> bit of trust issues regarding the new method, and I'd like to be able to >> test potential issues against a stock postgres by doing a normal i

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Michael Paquier
On Tue, Aug 6, 2013 at 12:41 PM, Fujii Masao wrote: >> - >> + unlink(PROMOTE_SIGNAL_FILE); >> Wouldn't it make sense to keep the call to stat() to check the file >> status before unlinking it? > > Why do we need to check the existence of the file before removing it > here? Forget wha

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Andres Freund
Fujii Masao schrieb: >On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund >wrote: >> Hi, >> >> On 2013-08-06 03:24:58 +0900, Fujii Masao wrote: >>> Hi all, >>> >>> We discussed the $SUBJECT in the following threads: >>> >http://www.postgresql.org/message-id/ca+tgmozbr+wl8e7mf_krp6fy4fd2pmr11tpiuyjmf

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Fujii Masao
On Tue, Aug 6, 2013 at 11:40 AM, Andres Freund wrote: > Hi, > > On 2013-08-06 03:24:58 +0900, Fujii Masao wrote: >> Hi all, >> >> We discussed the $SUBJECT in the following threads: >> http://www.postgresql.org/message-id/ca+tgmozbr+wl8e7mf_krp6fy4fd2pmr11tpiuyjmfx_vtg1...@mail.gmail.com >> http:/

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Fujii Masao
On Tue, Aug 6, 2013 at 11:20 AM, Michael Paquier wrote: > On Tue, Aug 6, 2013 at 3:24 AM, Fujii Masao wrote: >> Hi all, >> >> We discussed the $SUBJECT in the following threads: >> http://www.postgresql.org/message-id/ca+tgmozbr+wl8e7mf_krp6fy4fd2pmr11tpiuyjmfx_vtg1...@mail.gmail.com >> http://ww

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Andres Freund
Hi, On 2013-08-06 03:24:58 +0900, Fujii Masao wrote: > Hi all, > > We discussed the $SUBJECT in the following threads: > http://www.postgresql.org/message-id/ca+tgmozbr+wl8e7mf_krp6fy4fd2pmr11tpiuyjmfx_vtg1...@mail.gmail.com > http://www.postgresql.org/message-id/CAHGQGwEBUvgcx8X+Z0Hh+VdwYcJ8KCuR

Re: [HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Michael Paquier
On Tue, Aug 6, 2013 at 3:24 AM, Fujii Masao wrote: > Hi all, > > We discussed the $SUBJECT in the following threads: > http://www.postgresql.org/message-id/ca+tgmozbr+wl8e7mf_krp6fy4fd2pmr11tpiuyjmfx_vtg1...@mail.gmail.com > http://www.postgresql.org/message-id/CAHGQGwEBUvgcx8X+Z0Hh+VdwYcJ8KCuRuLt

[HACKERS] Should we remove "not fast" promotion at all?

2013-08-05 Thread Fujii Masao
Hi all, We discussed the $SUBJECT in the following threads: http://www.postgresql.org/message-id/ca+tgmozbr+wl8e7mf_krp6fy4fd2pmr11tpiuyjmfx_vtg1...@mail.gmail.com http://www.postgresql.org/message-id/CAHGQGwEBUvgcx8X+Z0Hh+VdwYcJ8KCuRuLt1jSsxeLxPcX=0...@mail.gmail.com Our consensus seems to remov