Talk About Network

Google


Register and Login
Nick
Password
Register create new account Sign up is FREE and you can post replies, new topics, bookmark posts and more!
Recover lost password


Data Bases > Pgsql Hackers > Re: Fairly seri...
Latest [ Topics | Posts ] Archive Post A New Topic Post a Reply
<< Topic < Post Post 15 of 17 Topic 9348 of 11009
Post > Topic >>

Re: Fairly serious bug induced by latest guc enum changes

by magnus@[EMAIL PROTECTED] (Magnus Hagander) Jul 1, 2008 at 09:10 PM

Tom Lane wrote:
> Magnus Hagander <magnus@[EMAIL PROTECTED]
> writes:
>> Or are you talking about changing the variable "fsync"? If so, doesn't
>> "fsync=off" also change the behavior of other parts of the code, so
it's
>> not just WAL, which means it'd be pretty unsafe *anyway* unless you
>> actually "sync" the disks, and not just fsync?
> 
> No, because the other uses of it are controlling whether to issue
> fsync() calls dynamically.  The use in get_sync_bit is the only one
> that sets persistent state.  In fact md.c goes out of its way to ensure
> that changing fsync on the fly behaves as expected.

Not having looked at md.c (I confess...) but don't we have a problem in
case we have closed the file without fsyncing it, and then change the
fsync parameter?

Either way, I see your point, but I doubt it's worth getting upset over.
Funning with fsync=off in the first place is bad, and if it takes you
one WAL segment to "recover", I think that's acceptable...

//Magnus


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@[EMAIL PROTECTED]
)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
 




 17 Posts in Topic:
Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-05-12 16:22:12 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-05-12 22:33:33 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-05-12 18:46:06 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-05-13 08:07:02 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-05-13 09:44:14 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-05-13 09:50:29 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-05-13 15:54:21 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-05-13 10:10:16 
Re: Fairly serious bug induced by latest guc enum
bruce@[EMAIL PROTECTED]   2008-06-30 16:29:48 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-06-30 16:41:12 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-07-01 20:06:00 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-07-01 14:51:21 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-07-01 20:57:38 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-07-01 15:07:21 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-07-01 21:10:34 
Re: Fairly serious bug induced by latest guc enum changes
tgl@[EMAIL PROTECTED] (T  2008-07-01 15:17:06 
Re: Fairly serious bug induced by latest guc enum changes
magnus@[EMAIL PROTECTED]   2008-07-02 14:11:21 

Post A Reply:
  Go here to Signup

AddThis Feed Button


About - Advertising - Contact - Frequently Asked Questions - Privacy Policy - Terms of Use - Signup

Contact
tan12V112 Fri Dec 5 8:53:02 CST 2008.