Do you have commit access to apply the patch yourself? I would be happy to review. I am in the middle of a testing cycle right now and am loath to touch anything... Jody > Hi, > > I uploaded a new patch to jira. However, there is one thing to > consider: wether there are DataStoreFactories, that needs to be > "available" for canProcess to work. Anyhow, it will work, in that case > too, but the warning messag would look different. > > Peter > > Jody Garnett wrote: >> That sounds perfect! We all get what we want >> - we don't want new users annoyed with warning from implementations >> that are not available (when they are not even "targeted") >> - we want feedback when arcsde is missing its jars and we intended to >> use it >> >> Thanks for the bug report; let's take our comments there. >> Jody >> >> Bolla Péter wrote: >>> Hi, >>> >>> As I checked the stack trace, it is comming from isAvailable(). >>> >>> Some month ago I proposed, that DataStoreFinder is doing things in >>> an inappropriate way. (Consuming important exception. See: >>> http://jira.codehaus.org/browse/GEOT-1403 ) Now I would add, that it >>> should be modified in a way, that check canProcess() for ALL >>> datastores (not only available ones), and report an error if a >>> datastore could process a connection, but it is not available. >>> >>> What do you think? >>> >>> Best, >>> Peter >>> >> >>
------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
