[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-05 Thread Donald Woods (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Donald Woods updated OPENJPA-1604: -- Patch Info: [Patch Available] Affects Version/s: 2.0.0 Fix Version/s: 2.1.0

[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-08 Thread Rick Curtis (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rick Curtis updated OPENJPA-1604: - Attachment: OPENJPA-1604-trunk.patch OPENJPA-1604-2.0.x.patch After digging int

[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-08 Thread Rick Curtis (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rick Curtis updated OPENJPA-1604: - Attachment: (was: OPENJPA-1604-trunk.patch) > Setting PessimisticLockManager fails to appen

[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-08 Thread Rick Curtis (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rick Curtis updated OPENJPA-1604: - Attachment: (was: OPENJPA-1604-2.0.x.patch) > Setting PessimisticLockManager fails to appen

[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-08 Thread Rick Curtis (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rick Curtis updated OPENJPA-1604: - Attachment: OPENJPA-1604-2.0.x.patch OPENJPA-1604-trunk.patch > Setting Pessimi

[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-10 Thread Donald Woods (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Donald Woods updated OPENJPA-1604: -- Fix Version/s: (was: 2.1.0) Affects Version/s: (was: 2.1.0) > Setting Pessimi

[jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-26 Thread Jeremy Bauer (JIRA)
[ https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jeremy Bauer updated OPENJPA-1604: -- Attachment: OPENJPA-1604_optimistic_fix_2.0.x.patch Attaching a patch for 2.0.x which takes i

Re: [jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-08 Thread Albert Lee
Please localized the trace message. Thanks Albert Lee. On Thu, Apr 8, 2010 at 2:07 PM, Rick Curtis (JIRA) wrote: > > [ > https://issues.apache.org/jira/browse/OPENJPA-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel] > > Rick Curtis updated OPENJPA-1604: >

Re: [jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-08 Thread Rick Curtis
Albert - When I committed the code I changed the trace string to localized warning messages, but this brings up a very good point. I know there is precedence for localizing trace messages, but does that mean it is the right thing to do? Personally, I like putting trace in my code so I can actuall

Re: [jira] Updated: (OPENJPA-1604) Setting PessimisticLockManager fails to append "for update clause" to the select statement

2010-04-09 Thread Kevin Sutter
Rick, I agree. I don't like localizing trace messages either. Trace is supposed to be meant for us developers to help debug a problem. If it gets localized to say French, it's no good to me. But, this should be a whole separate thread on the dev forum... :-) Thanks, Kevin On Thu, Apr 8, 2010