Thanks Scott. I see Leon has discovered a similar issue for a different SECA in Jira 4469, but his patch uses global-commit rather than return.
I'm happy to submit a patch, but would Leon's approach be better? Cheers Paul Foxworthy Scott Gray-2 wrote: > > +1 > > Regards > Scott > > On 12/10/2011, at 12:09 PM, Paul Foxworthy wrote: > >> Hi all, >> >> Last year the SECA that runs loadSalesInvoiceFact was changed to run >> async. >> See >> https://fisheye6.atlassian.com/browse/ofbiz/trunk/applications/accounting/servicedef/secas_olap.xml?hb=true#to29 >> . >> >> That makes sense to me, but it's run on commit, (that is, prior to the >> commit) and I'm getting errors in the log that the invoice doesn't exist. >> It >> seems that the async loadSalesInvoiceFact is sometimes running before the >> transaction is committed on another thread, so it can't see the invoice >> data. >> >> Should it be run on return rather than commit? >> >> Thanks >> >> Paul Foxworthy >> >> >> -- >> View this message in context: >> http://ofbiz.135035.n4.nabble.com/Should-loadSalesInvoiceFact-be-run-on-return-instead-of-commit-tp3896289p3896289.html >> Sent from the OFBiz - Dev mailing list archive at Nabble.com. > -- View this message in context: http://ofbiz.135035.n4.nabble.com/Should-loadSalesInvoiceFact-be-run-on-return-instead-of-commit-tp3896289p3897004.html Sent from the OFBiz - Dev mailing list archive at Nabble.com.