Emmanuel,
I'm applying the latest patch from Amin, the patch is fine but
thinking more on the async/sync I realized we forgot the other
implementation, so I added an ErrorHandler implementation which will
rethrow the exception up to the thread which committed the
transaction, and another test for this.
Could you check it, especially the configuration option key and names
for the two error handlers?
To close the issue, I'll wait for Amin to write a little documentation patch.
Sanne
2010/4/3 Sanne Grinovero <sanne.grinovero(a)gmail.com>:
2010/4/3 Emmanuel Bernard <emmanuel(a)hibernate.org>:
> Ah cool, we are getting there :)
> Looking at it quickly, can you use tabs, not space for indentation? That's our
coding conventions.
>
> One open question I had last time was one error handler for sync and async, vs one
for sync and one for async. What was decided? What was the reasoning?
>
Initially we thought that that would be needed, but doesn't appear to
be the case. Latest Amin's work did remove the duality, sync and async
are using the same methods now.