On 13/02/13 17:11, Andrea Aime wrote:
> On Wed, Feb 13, 2013 at 5:02 PM, Ákos Maróy <a...@maroy.hu
> <mailto:a...@maroy.hu>> wrote:
>
>> However.. this might be related, but not the same
>>
> I don't see the changes committed in relation to the above ticket
>
>
> Where did you look? :-)
> https://github.com/geotools/geotools/commit/3c7fcb062171bf62294a42fdcb74130082323a6f
thanks, I didn't find this link on the Jira ticket page
>
>
> OTOH, the issue I reported might be a bit different, as in this
> case, there is no cancellation request, but an exception is thrown
> which is not handled.
>
>
> Hum... the test to try and reproduce lockups is there in the above
> patch, can you modify
> it to reproduce this one as well?
to reproduce the issue, the execute() method of a RenderingRequest has
to throw a runtime exception, when called from within
PainterThread.run(). I looked at the test case in the above patch, and
there the exception is thrown within a direct call to
StreamingRenderer.paint(), not from within a RenderingRequest
I looked at the code, and it's not obvious for me on how to inject a
special RenderingRequest into the requests queue, that would throw a
runtime exception in its execute() method :(
> Anyways, jira tickets and patches to fix the issue welcomed
>
would be glad to create a patch with a test included, but see above, I
don't see how to easily inject a RenderingRequest that throws a runtime
exception
------------------------------------------------------------------------------
Free Next-Gen Firewall Hardware Offer
Buy your Sophos next-gen firewall before the end March 2013
and get the hardware for free! Learn more.
http://p.sf.net/sfu/sophos-d2d-feb
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
