I noticed this block of code: https://github.com/geoserver/geoserver/blob/master/src/web/demo/src/main/java/org/geoserver/web/catalogstresstool/CatalogStressTester.java#L276-287
(Not because I’m doing a thorough review, it just happens to fail compilation against Wicket 1.5)
I notice that “startLink” and “progress” are being added to the AjaxRequestTarget both before the try-block and in its finally clause. This seems unnecessary - what’s going on there?
While I’m at it, is there a more specific exception type than “Exception” that we should be catching here?
–
David Winslow
OpenGeo - http://opengeo.org/