neil.ellis at peepwl.com
Thu Jul 30 22:07:29 CEST 2009
Thanks as ever Johan.
On 30 Jul 2009, at 20:03, Johan Svensson wrote:
> You are right, we should change that generic one to a specific runtime
> exception. Will look into that during our exception refactoring.
> On Thu, Jul 30, 2009 at 8:51 PM, Neil Ellis<neil.ellis at peepwl.com>
>> Hi Johan
>> I meant the application is being shutdown (not neo.shutdown()) :-) -
>> though I understand the confusion. So as part of our shutdown process
>> we will tell all active threads to stop by interrupting them. When
>> they've all stopped I'll shutdown Neo.
>> So the thread this code runs in is receiving an interrupt, however
>> is converting the exception into a generic RuntimeException. Which
>> makes it very hard to catch, with the code being like this it makes
>> very hard for me to request a thread interruption and then deal with
>> the consequences (i.e. a RuntimeException does not tell me that the
>> thread has been interrupted, so I don't know whether to report an
>> error and continue - or to end the loop it is in).
>> Thanks as always
>> On 30 Jul 2009, at 19:45, Johan Svensson wrote:
>>> Hi Neil,
>>> Yes, there are a lot of error messages that needs to be improved. On
>>> the other hand the documentation should state that calling any Neo4j
>>> API operation after (or concurrently with) NeoService.shutdown() is
>>> not valid. Adding a flag to check if running is not an option
>>> of performance issues. Could be that there are certain places we
>>> throw a better exception with a better message to help the user
>>> down what is wrong. We are planing to do a refactoring of the low
>>> level exceptions before 1.0 final and we will keep this in mind
>>> On Thu, Jul 23, 2009 at 2:14 AM, Neil
>>> Ellis<neil.ellis at mangala.co.uk> wrote:
>>>> Hi guys, if a thread gets interrupted (i.e. during a shutdown), I
>>>> up with a RuntimeException, I think it would be better to bubble
>>>> something meaningful. What do you think?
>>>> org.neo4j.impl.nioneo.store.StoreFailureException: Unable to load
>>>> position @...
> Neo mailing list
> User at lists.neo4j.org
More information about the User