[hibernate-dev] release plans for Search?
Amin Mohammed-Coleman
aminmc at gmail.com
Tue Apr 6 05:45:23 EDT 2010
Hi
I'll create a patch for the documentation to minor this. The current
documentation patch talks about the rethrow.
Cheers
Amin
Sent from my iPhone
On 6 Apr 2010, at 09:31, Emmanuel Bernard <emmanuel at hibernate.org>
wrote:
> Let's remove it for now.
> Re-throw should never be the default even when we have that fixed:
> database first, index optional is the best default.
>
> On 5 avr. 2010, at 17:17, Sanne Grinovero wrote:
>
>> Ah right, I nearly forgot that - overconfidence from my incomplete
>> test.
>> Shall I completely remove the implementation or just comment on it
>> and
>> remove it from being a default?
>>
>> Sanne
>>
>> 2010/4/5 Emmanuel Bernard <emmanuel at hibernate.org>:
>>> The problem with the exception impl is that it does not work today
>>> because Core's Synch does swallow exceptions. See "Exceptions
>>> thrown in a tx synchronization are eaten".
>>> I am not sure we should put it in until this is resolved.
>>>
>>> On 4 avr. 2010, at 18:34, Sanne Grinovero wrote:
>>>
>>>> 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 at gmail.com>:
>>>>> 2010/4/3 Emmanuel Bernard <emmanuel at 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.
>>>>>
>>>
>>>
>
More information about the hibernate-dev
mailing list