OK Just Oracle failing now - I'll need to spin up an oracle vm (or just install it on my laptop) tomorrow to test it,
Ian On Mon, Jun 1, 2015 at 12:22 PM Ian Turton <[email protected]> wrote: > OK I'll push them this afternoon > > Ian > > On Mon, 1 Jun 2015 11:15 am Andrea Aime <[email protected]> > wrote: > >> Hi Ian, >> those are all related to a mix of jdbc driver and postgres version >> issues, don't worry, >> you haven't introduced them >> >> Cheers >> Andrea >> >> >> On Mon, Jun 1, 2015 at 12:12 PM, Ian Turton <[email protected]> wrote: >> >>> OK I think I've fixed the bug I introduced - I wasn't recreating the >>> dates table in the default timezone after my test. >>> >>> I'm now left with 3 errors else where! These might be related to >>> upgrading my jdbc driver to avoid LOB failures? Has anyone seen similar? >>> >>> PostgreSQL 9.4.2 on x86_64-unknown-linux-gnu, compiled by gcc (Ubuntu >>> 4.8.2-19ubuntu1) 4.8.2, 64-bit >>> -------------- >>> POSTGIS="2.1.7 r13414" GEOS="3.4.2-CAPI-1.8.2 r3921" PROJ="Rel. 4.8.0, >>> 6 March 2012" GDAL="GDAL 1.10.1, released 2013/08/26" LIBXML="2.9.1" >>> LIBJSON="UNKNOWN" TOPOLOGY RASTER >>> >>> Failed tests: >>> >>> PostGISFunctionOnlineTest>OnlineTestCase.run:123->JDBCFunctionOnlineTest.testStrEndsWithOtherProperty:101->JDBCFunctionOnlineTest.assertFeatures:319 >>> Expected to find 3 features, but was 1 >>> >>> Tests in error: >>> >>> PostgisNGDataStoreFactoryOnlineTest>OnlineTestCase.run:123->testCreateConnection:53->checkCreateConnection:81 >>> » Runtime >>> >>> PostgisNGDataStoreFactoryOnlineTest>OnlineTestCase.run:123->testCreateConnectionWithOldId:58->checkCreateConnection:81 >>> » Runtime >>> >>> >>> On Fri, May 29, 2015 at 2:36 PM Ian Turton <[email protected]> wrote: >>> >>>> This is a harder fix than I first thought, very hard to debug as the >>>> tests pass individually, >>>> >>>> I did find why my local build missed it though - some one changed the >>>> PostGIS port for Postgresql 9.4 from 5432 so my fixture failed quietly. :-( >>>> >>>> Any way I'll keep looking to see what's up with PostGIS >>>> >>>> Ian >>>> >>>> >>>> >>>> On Fri, May 29, 2015 at 9:55 AM Ian Turton <[email protected]> wrote: >>>> >>>>> That's weird, I was sure I ran against postgis on my local build. >>>>> I'll look at the issue this morning (once I make my network work again), >>>>> and try fixing it. It's a bit worrying that the tests don't fail >>>>> consistently. >>>>> >>>>> Ian >>>>> >>>>> On Fri, 29 May 2015 12:43 am Ben Caradoc-Davies <[email protected]> >>>>> wrote: >>>>> >>>>>> Oh, much breakage: >>>>>> >>>>>> https://osgeo-org.atlassian.net/browse/GEOT-5047 >>>>>> https://github.com/geotools/geotools/pull/858 >>>>>> >>>>>> Breaks online tests for gt-jdbc-oracle and gt-jdbc-sqlserver on >>>>>> Boundless Jenkins: >>>>>> >>>>>> http://ares.boundlessgeo.com/jenkins/view/geotools/job/geotools-master-online/134/consoleText >>>>>> >>>>>> Breaks gt-jdbc-postgis for me locally against postgres 9.4 / postgis >>>>>> 2.1: >>>>>> PostgisUDTOnlineTest>OnlineTestCase.run:123->testRead:81 >>>>>> expected:<2004-10-30 1[7]:30:00.0> but was:<2004-10-30 1[8]:30:00.0> >>>>>> >>>>>> org.geotools.data.postgis.PostgisUDTOnlineTest passes when run by >>>>>> itself >>>>>> in Maven or Eclipse. Leaky fixture? >>>>>> >>>>>> All tests in (ps and non-ps) PostgisTemporalFilterOnlineTest fail in >>>>>> Eclipse when all tests for the module are run. They pass when run >>>>>> individually. >>>>>> >>>>>> No breakage in GeoServer app-schema online tests against postgis. >>>>>> >>>>>> Kind regards, >>>>>> Ben. >>>>>> >>>>>> On 28/05/15 20:27, Ian Turton wrote: >>>>>> > Currently the only place I know that changes them at runtime is in >>>>>> the new >>>>>> > test :-) >>>>>> > >>>>>> > Good catch on the static DateFormaters - I remember being bitten by >>>>>> that in >>>>>> > the past, now. I'll refactor that out this morning. >>>>>> > >>>>>> > Ian >>>>>> > >>>>>> > On Thu, May 28, 2015 at 9:23 AM Andrea Aime < >>>>>> [email protected]> >>>>>> > wrote: >>>>>> > >>>>>> >> Hi Ian, >>>>>> >> sorry I could not review in time, but it looks good. Curious about >>>>>> the >>>>>> >> timezone changing at runtime, which software does that? >>>>>> >> >>>>>> >> Actually wait a second, that section of code is buggy in another >>>>>> way, one >>>>>> >> cannot have statically cached date formatters, they are not >>>>>> >> thread safe... >>>>>> >> >>>>>> >> Cheers >>>>>> >> Andrea >>>>>> >> >>>>>> >> On Thu, May 28, 2015 at 10:18 AM, Ian Turton <[email protected]> >>>>>> wrote: >>>>>> >> >>>>>> >>> I've fixed my local build now and tested a local GeoServer build >>>>>> against >>>>>> >>> it too. So I've gone a head and merged. >>>>>> >>> >>>>>> >>> Ian >>>>>> >>> >>>>>> >>> On Tue, May 26, 2015 at 5:59 PM Ian Turton <[email protected]> >>>>>> wrote: >>>>>> >>> >>>>>> >>>> There is now a PR that should solve this problem - >>>>>> >>>> https://github.com/geotools/geotools/pull/858 I'm waiting on >>>>>> Travis as >>>>>> >>>> my mchine is (wierdly) refusing to build either my branch or >>>>>> master due >>>>>> >>>> to missing Assertions in gt-coverage . So if someone else can >>>>>> check it and >>>>>> >>>> merge that would be great. >>>>>> >>>> >>>>>> >>>> >>>>>> >>>> Ian >>>>>> >>>> >>>>>> >>>> On Sat, May 16, 2015 at 7:26 PM Ian Turton <[email protected]> >>>>>> wrote: >>>>>> >>>> >>>>>> >>>>> Not yet but now I'm at loose end and back on the Internet at the >>>>>> >>>>> weekend I should have a good chance to fix it next week. >>>>>> >>>>> >>>>>> >>>>> Ian >>>>>> >>>>> >>>>>> >>>>> On Sat, 16 May 2015 6:04 pm Andrea Aime < >>>>>> [email protected]> >>>>>> >>>>> wrote: >>>>>> >>>>> >>>>>> >>>>>> On Sun, Apr 19, 2015 at 4:06 PM, Ian Turton < >>>>>> [email protected]> >>>>>> >>>>>> wrote: >>>>>> >>>>>> >>>>>> >>>>>>> That all makes a lot of sense, I was never clear on why we >>>>>> coerced >>>>>> >>>>>>> them into GMT. >>>>>> >>>>>>> >>>>>> >>>>>>> That leaves the JDBC date test case - I think that if we stop >>>>>> forcing >>>>>> >>>>>>> dates into GMT then it will go back to working east of >>>>>> Greenwich as the DB >>>>>> >>>>>>> will be created in the local time zone - I'll try and have a >>>>>> look this week. >>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> >>>>>> Hi Ian, >>>>>> >>>>>> did you ever make progress on this one? >>>>>> >>>>>> >>>>>> >>>>>> Cheers >>>>>> >>>>>> Andrea >>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> >>>>>> == >>>>>> >>>>>> GeoServer Professional Services from the experts! Visit >>>>>> >>>>>> http://goo.gl/NWWaa2 for more information. >>>>>> >>>>>> == >>>>>> >>>>>> >>>>>> >>>>>> Ing. Andrea Aime >>>>>> >>>>>> @geowolf >>>>>> >>>>>> Technical Lead >>>>>> >>>>>> >>>>>> >>>>>> GeoSolutions S.A.S. >>>>>> >>>>>> Via Poggio alle Viti 1187 >>>>>> >>>>>> 55054 Massarosa (LU) >>>>>> >>>>>> Italy >>>>>> >>>>>> phone: +39 0584 962313 >>>>>> >>>>>> fax: +39 0584 1660272 >>>>>> >>>>>> mob: +39 339 8844549 >>>>>> >>>>>> >>>>>> >>>>>> http://www.geo-solutions.it >>>>>> >>>>>> http://twitter.com/geosolutions_it >>>>>> >>>>>> >>>>>> >>>>>> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003* >>>>>> >>>>>> >>>>>> >>>>>> Le informazioni contenute in questo messaggio di posta >>>>>> elettronica e/o >>>>>> >>>>>> nel/i file/s allegato/i sono da considerarsi strettamente >>>>>> riservate. Il >>>>>> >>>>>> loro utilizzo è consentito esclusivamente al destinatario del >>>>>> messaggio, >>>>>> >>>>>> per le finalità indicate nel messaggio stesso. Qualora >>>>>> riceviate questo >>>>>> >>>>>> messaggio senza esserne il destinatario, Vi preghiamo >>>>>> cortesemente di >>>>>> >>>>>> darcene notizia via e-mail e di procedere alla distruzione del >>>>>> messaggio >>>>>> >>>>>> stesso, cancellandolo dal Vostro sistema. Conservare il >>>>>> messaggio stesso, >>>>>> >>>>>> divulgarlo anche in parte, distribuirlo ad altri soggetti, >>>>>> copiarlo, od >>>>>> >>>>>> utilizzarlo per finalità diverse, costituisce comportamento >>>>>> contrario ai >>>>>> >>>>>> principi dettati dal D.Lgs. 196/2003. >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> The information in this message and/or attachments, is >>>>>> intended solely >>>>>> >>>>>> for the attention and use of the named addressee(s) and may be >>>>>> confidential >>>>>> >>>>>> or proprietary in nature or covered by the provisions of >>>>>> privacy act >>>>>> >>>>>> (Legislative Decree June, 30 2003, no.196 - Italy's New Data >>>>>> Protection >>>>>> >>>>>> Code).Any use not in accord with its purpose, any disclosure, >>>>>> reproduction, >>>>>> >>>>>> copying, distribution, or either dissemination, either whole >>>>>> or partial, is >>>>>> >>>>>> strictly forbidden except previous formal approval of the named >>>>>> >>>>>> addressee(s). If you are not the intended recipient, please >>>>>> contact >>>>>> >>>>>> immediately the sender by telephone, fax or e-mail and delete >>>>>> the >>>>>> >>>>>> information in this message that has been received in error. >>>>>> The sender >>>>>> >>>>>> does not give any warranty or accept liability as the content, >>>>>> accuracy or >>>>>> >>>>>> completeness of sent messages and accepts no responsibility >>>>>> for changes >>>>>> >>>>>> made after they were sent or for other risks which arise as a >>>>>> result of >>>>>> >>>>>> e-mail transmission, viruses, etc. >>>>>> >>>>>> >>>>>> >>>>>> ------------------------------------------------------- >>>>>> >>>>>> >>>>>> >>>>> >>>>>> >> >>>>>> >> >>>>>> >> -- >>>>>> >> == >>>>>> >> Meet us at the INSPIRE Conference in Lisbon 25-29 May 2015! Visit >>>>>> >> http://goo.gl/WHKDXT for more information. >>>>>> >> == >>>>>> >> >>>>>> >> Ing. Andrea Aime >>>>>> >> @geowolf >>>>>> >> Technical Lead >>>>>> >> >>>>>> >> GeoSolutions S.A.S. >>>>>> >> Via Poggio alle Viti 1187 >>>>>> >> 55054 Massarosa (LU) >>>>>> >> Italy >>>>>> >> phone: +39 0584 962313 >>>>>> >> fax: +39 0584 1660272 >>>>>> >> mob: +39 339 8844549 >>>>>> >> >>>>>> >> http://www.geo-solutions.it >>>>>> >> http://twitter.com/geosolutions_it >>>>>> >> >>>>>> >> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003* >>>>>> >> >>>>>> >> Le informazioni contenute in questo messaggio di posta elettronica >>>>>> e/o >>>>>> >> nel/i file/s allegato/i sono da considerarsi strettamente >>>>>> riservate. Il >>>>>> >> loro utilizzo è consentito esclusivamente al destinatario del >>>>>> messaggio, >>>>>> >> per le finalità indicate nel messaggio stesso. Qualora riceviate >>>>>> questo >>>>>> >> messaggio senza esserne il destinatario, Vi preghiamo cortesemente >>>>>> di >>>>>> >> darcene notizia via e-mail e di procedere alla distruzione del >>>>>> messaggio >>>>>> >> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio >>>>>> stesso, >>>>>> >> divulgarlo anche in parte, distribuirlo ad altri soggetti, >>>>>> copiarlo, od >>>>>> >> utilizzarlo per finalità diverse, costituisce comportamento >>>>>> contrario ai >>>>>> >> principi dettati dal D.Lgs. 196/2003. >>>>>> >> >>>>>> >> >>>>>> >> >>>>>> >> The information in this message and/or attachments, is intended >>>>>> solely for >>>>>> >> the attention and use of the named addressee(s) and may be >>>>>> confidential or >>>>>> >> proprietary in nature or covered by the provisions of privacy act >>>>>> >> (Legislative Decree June, 30 2003, no.196 - Italy's New Data >>>>>> Protection >>>>>> >> Code).Any use not in accord with its purpose, any disclosure, >>>>>> reproduction, >>>>>> >> copying, distribution, or either dissemination, either whole or >>>>>> partial, is >>>>>> >> strictly forbidden except previous formal approval of the named >>>>>> >> addressee(s). If you are not the intended recipient, please contact >>>>>> >> immediately the sender by telephone, fax or e-mail and delete the >>>>>> >> information in this message that has been received in error. The >>>>>> sender >>>>>> >> does not give any warranty or accept liability as the content, >>>>>> accuracy or >>>>>> >> completeness of sent messages and accepts no responsibility for >>>>>> changes >>>>>> >> made after they were sent or for other risks which arise as a >>>>>> result of >>>>>> >> e-mail transmission, viruses, etc. >>>>>> >> >>>>>> >> ------------------------------------------------------- >>>>>> >> >>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>> ------------------------------------------------------------------------------ >>>>>> > >>>>>> > >>>>>> > >>>>>> > _______________________________________________ >>>>>> > GeoTools-Devel mailing list >>>>>> > [email protected] >>>>>> > https://lists.sourceforge.net/lists/listinfo/geotools-devel >>>>>> > >>>>>> >>>>>> -- >>>>>> Ben Caradoc-Davies <[email protected]> >>>>>> Director >>>>>> Transient Software Limited <http://transient.nz/> >>>>>> New Zealand >>>>>> >>>>> >> >> >> -- >> == >> GeoServer Professional Services from the experts! Visit >> http://goo.gl/it488V for more information. >> == >> >> Ing. Andrea Aime >> @geowolf >> Technical Lead >> >> GeoSolutions S.A.S. >> Via Poggio alle Viti 1187 >> 55054 Massarosa (LU) >> Italy >> phone: +39 0584 962313 >> fax: +39 0584 1660272 >> mob: +39 339 8844549 >> >> http://www.geo-solutions.it >> http://twitter.com/geosolutions_it >> >> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003* >> >> Le informazioni contenute in questo messaggio di posta elettronica e/o >> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il >> loro utilizzo è consentito esclusivamente al destinatario del messaggio, >> per le finalità indicate nel messaggio stesso. Qualora riceviate questo >> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di >> darcene notizia via e-mail e di procedere alla distruzione del messaggio >> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso, >> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od >> utilizzarlo per finalità diverse, costituisce comportamento contrario ai >> principi dettati dal D.Lgs. 196/2003. >> >> >> >> The information in this message and/or attachments, is intended solely >> for the attention and use of the named addressee(s) and may be confidential >> or proprietary in nature or covered by the provisions of privacy act >> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection >> Code).Any use not in accord with its purpose, any disclosure, reproduction, >> copying, distribution, or either dissemination, either whole or partial, is >> strictly forbidden except previous formal approval of the named >> addressee(s). If you are not the intended recipient, please contact >> immediately the sender by telephone, fax or e-mail and delete the >> information in this message that has been received in error. The sender >> does not give any warranty or accept liability as the content, accuracy or >> completeness of sent messages and accepts no responsibility for changes >> made after they were sent or for other risks which arise as a result of >> e-mail transmission, viruses, etc. >> >> ------------------------------------------------------- >> >
------------------------------------------------------------------------------
_______________________________________________ GeoTools-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
