Wouldn't it be reasonable to expect users of putForExternalRead() to use SKIP_LOCK combined with it? On 31 October 2012 16:05, Manik Surtani <manik(a)jboss.org&gt; wrote: > Guys > > Someone's reported https://issues.jboss.org/browse/ISPN-2426 which can cause a stale lock when using putForExternalRead() in a non-transactional context. > > The contributed patch looks good, I'm submitting a pull request - but a question to the Hibernate and AS folks on the list, I presume this is something you are both exposed to? Or do you only use putForExternalRead() in a transactional context? > > Cheers > Manik > -- > Manik Surtani > manik(a)jboss.org > twitter.com/maniksurtani > > Platform Architect, JBoss Data Grid > http://red.ht/data-grid > > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev

On Oct 31, 2012, at 4:05 PM, Manik Surtani <manik(a)jboss.org&gt; wrote: > Guys > > Someone's reported https://issues.jboss.org/browse/ISPN-2426 which can cause a stale lock when using putForExternalRead() in a non-transactional context. > > The contributed patch looks good, I'm submitting a pull request - but a question to the Hibernate and AS folks on the list, I presume this is something you are both exposed to? Or do you only use putForExternalRead() in a transactional context? For the current 2LC, PFER is used within a transactional context.

> > Cheers > Manik > -- > Manik Surtani > manik(a)jboss.org > twitter.com/maniksurtani > > Platform Architect, JBoss Data Grid > http://red.ht/data-grid > > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Galder Zamarreño galder(a)redhat.com twitter.com/galderz Project Lead, Escalante http://escalante.io Engineer, Infinispan http://infinispan.org _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev

On Nov 2, 2012, at 1:13 PM, Manik Surtani <manik(a)jboss.org&gt; wrote: > > On 1 Nov 2012, at 16:06, Galder Zamarreño <galder(a)redhat.com&gt; wrote: > >> >> >> On Oct 31, 2012, at 4:05 PM, Manik Surtani <manik(a)jboss.org&gt; wrote: >> >>> Guys >>> >>> Someone's reported https://issues.jboss.org/browse/ISPN-2426 which can cause a stale lock when using putForExternalRead() in a non-transactional context. >>> >>> The contributed patch looks good, I'm submitting a pull request - but a question to the Hibernate and AS folks on the list, I presume this is something you are both exposed to? Or do you only use putForExternalRead() in a transactional context? >> >> For the current 2LC, PFER is used within a transactional context. > > Always? Actually it's both. For read only entities, you could execute a PFER outside of a transaction. The other mode is transactional, in which case PFER is transactional. ISPN-2426 has been fixed in 5.1.x as well, so should feed into the product.

> >> >>> >>> Cheers >>> Manik >>> -- >>> Manik Surtani >>> manik(a)jboss.org >>> twitter.com/maniksurtani >>> >>> Platform Architect, JBoss Data Grid >>> http://red.ht/data-grid >>> >>> >>> _______________________________________________ >>> infinispan-dev mailing list >>> infinispan-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/infinispan-dev >> >> >> -- >> Galder Zamarreño >> galder(a)redhat.com >> twitter.com/galderz >> >> Project Lead, Escalante >> http://escalante.io >> >> Engineer, Infinispan >> http://infinispan.org >> >> >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > -- > Manik Surtani > manik(a)jboss.org > twitter.com/maniksurtani > > Platform Architect, JBoss Data Grid > http://red.ht/data-grid > > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Galder Zamarreño galder(a)redhat.com twitter.com/galderz Project Lead, Escalante http://escalante.io Engineer, Infinispan http://infinispan.org _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev