Ciao Ben, I waited a bit aside since I did not want to add another workaround in the ImageMosaic, but yeah, I would say go ahead with the merge.
Regards, Simone Giannecchini == Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information. == Ing. Simone Giannecchini @simogeo Founder/Director GeoSolutions S.A.S. Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 962313 mob: +39 333 8128928 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- On Thu, Sep 27, 2012 at 9:22 AM, Ben Caradoc-Davies <ben.caradoc-dav...@csiro.au> wrote: > On 20/09/12 10:44, Ben Caradoc-Davies wrote: >> >> On 19/09/12 17:16, Andrea Aime wrote: >>> >>> Ben, the fix seems indeed quite easy, why don't you give it a shot? :-p >> >> I tried the easy fix on Tuesday (see patch attached to the issue) but it >> broke the test. :-( >> I will have another look if I have time. > > > Done! I found the second spot in imagemosaic where special handling occurs > for H2 connection URLs. My updated patch ensures that jdbc URLs are now fed > to H2 in the form it expects: without percent encoding. > > I am now shopping for a reviewer: > > [GEOT-4262] imagemosaic incorrect handling of H2 database paths with spaces > https://jira.codehaus.org/browse/GEOT-4262 > > Kind regards, > > > -- > Ben Caradoc-Davies <ben.caradoc-dav...@csiro.au> > Software Engineer > CSIRO Earth Science and Resource Engineering > Australian Resources Research Centre > > ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://ad.doubleclick.net/clk;258768047;13503038;j? http://info.appdynamics.com/FreeJavaPerformanceDownload.html _______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
