Home CVS Mailinglists Documentation Download
[Slony1-bugs] [Bug 218] slon often has to retry transactions due to concurrent update in 2.1.0.b2
bugzilla-daemon at main.slony.info bugzilla-daemon at main.slony.info
Wed Aug 10 15:13:03 PDT 2011 
http://www.slony.info/bugzilla/show_bug.cgi?id=218

Christopher Browne <cbbrowne at ca.afilias.info> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|cbbrowne at ca.afilias.info    |ssinger at ca.afilias.info

--- Comment #25 from Christopher Browne <cbbrowne at ca.afilias.info> 2011-08-10 15:13:03 PDT ---
(In reply to comment #24)
> Chris please my patch to fix the locking ordering. You can also see that commit
> https://github.com/ssinger/slony1-engine/commit/6cf2ba44eccdd10de07d64e8526ed5bcb1cd80c0

Ah, quite right.

It's easy to see all the cases, and with the patch, they are now all consistent
in locking sl_event_lock before sl_config_lock.

slonik/slonpostgres at cbbrowne [06:11:41] [~/slony1-engine.github/src] [master *]
-> % grep sl_config */*.c | grep sl_event
slon/remote_worker.c:                                                          
                 "lock table %s.sl_event_lock,%s.sl_config_lock;",
slon/remote_worker.c:                                                          
 "lock table %s.sl_event_lock,%s.sl_config_lock;"
slon/remote_worker.c:                                                          
 "lock table %s.sl_event_lock, %s.sl_config_lock;"
slon/remote_worker.c:                                                          
 "lock table %s.sl_event_lock,%s.sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonpostgres at cbbrowne [06:11:41] [~/slony1-engine.github/src] [master *]
-> % grep sl_config */*.c | grep sl_event
zsh: correct 'sl_event' to 'slevent' [nyae]? n
slon/remote_worker.c:                                                          
                 "lock table %s.sl_event_lock,%s.sl_config_lock;",
slon/remote_worker.c:                                                          
 "lock table %s.sl_event_lock,%s.sl_config_lock;"
slon/remote_worker.c:                                                          
 "lock table %s.sl_event_lock, %s.sl_config_lock;"
slon/remote_worker.c:                                                          
 "lock table %s.sl_event_lock,%s.sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;",
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;",
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
ik.c:                                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;",
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;",
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
ik.c:                                 "lock table \"_%s\".sl_event_lock,
\"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;",
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;",
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"
slonik/slonik.c:                                 "lock table
\"_%s\".sl_event_lock, \"_%s\".sl_config_lock;"

-- 
Configure bugmail: http://www.slony.info/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the Slony1-bugs mailing list

Content © 2007 Slony Development Group - Hosting provided by Command Prompt, Inc.