Hi Please find updated patch with the removal of the enum singleton.
Thanks Amin
On 17 Mar 2010, at 12:58, Sanne Grinovero wrote: > Hi Amin, > thanks for the update, see some thoughts: > > 2010/3/16 Amin Mohammed-Coleman <ami...@gmail.com>: >> Hi folks, >> >> I've removed the enum singleton and created a class(BackendExceptionHandler) >> which has 2 methods: >> >> >> public Thread.UncaughtExceptionHandler >> getUncaughtExceptionHandler(SearchConfiguration configuration); >> >> public boolean logException(SearchConfiguration configuration) > > Care to explain how I should use them? Are we not going to have a > common interface? In that case does it make sense to have a method > named "logException", which would imply a logging implementation? > >> >> The issue that I'm looking at is getting the search configuration to the >> methods. In order to get the SearchConfiguration to the methods defined in >> BackendExceptionHandler, I have to thread SearchFactoryImplementor. >> >> Using this approach I will have to define a method to get the search >> configuration from the SearchFactoryImplementor. I'm guess this isn't the >> best approach as this requires a significant change. I don't know what >> peoples thoughts are on this. I'm looking to set the >> BackendExceptionHandler up when the SearchFactory is created and then use >> it. Is there any currently approach that does something similar? > > Ah there's a subtle problem with that, which is that we can't hold a > reference in Search to a Configuration at runtime: use it at > SearchFactory creation, extract all what you need, but then you have > to clear references or we're going to have a memory leak. [1] > The solution is, as we do with all components, to start them during > SearchFactory initialization and then expose a getter to initialized > service. > > [1] http://opensource.atlassian.com/projects/hibernate/browse/HSEARCH-314 > > Cheers and thanks for the complex work, > Sanne > >> >> >> Thanks for your patience with this one. >> >> Cheers >> Amin >> >> >> >> >> On 9 Mar 2010, at 13:00, Amin Mohammed-Coleman wrote: >> >>> Hi Sanne >>> >>> You are right and Im not happy with the enum class. I wanted to have a >>> single configuration that was available on the creation of the search >>> factory and re use when required. I'll take a look at changing that with a >>> better solution. >>> >>> Cheers >>> >>> Amin >>> >>> Sent from my iPhone >>> >>> On 9 Mar 2010, at 12:47, Sanne Grinovero <sanne.grinov...@gmail.com> wrote: >>> >>>> yes that is it; >>>> There has been some talking about other strategies as well on previous >>>> mails, like jms, but that lead to nowhere so yes I'm suggesting now to >>>> forget about other default implementations at the moment and proceed >>>> as you just said. >>>> >>>> 2010/3/9 Emmanuel Bernard <emman...@hibernate.org>: >>>>> I thought the goal was to have something pluggable with two default impls >>>>> (exception and log). If that's what you are describing I am ok, if not, >>>>> then I don't understand ;) >>>>> >>>>> On 9 mars 2010, at 12:08, Sanne Grinovero wrote: >>>>> >>>>>> Emmanuel are you ok with this if we either log or rethrow the >>>>>> exception back to application code? jms et al looks complex and I >>>>>> don't see the benefit. >>>>> >>>>> >> >>
_______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev