Re: [Geotools-devel] Postgis 2.0 raster and postgres jdbc driver version

2012-10-14 Thread Ben Caradoc-Davies
That is also in the Jira issue!
https://jira.codehaus.org/browse/GEOT-4040

If we upgrade the driver, do we fix it for everyone?

On 12/10/12 21:30, christian.muel...@nvoe.at wrote:
> Hi Justin
>
> I found another solution:
>
> If you use a postgres driver lower than version 9 you can prepare your
> database
> with
>
> ALTER DATABASE dbname SET bytea_output TO 'escape';
>
> I will kick out my  tag, prepare my test database and add
> this info to the user guide. I think this is the cleanest solution.
>
>
> Zitat von Justin Deoliveira:
>
>> I would use a property to make this configurable. Indeed I ran into this
>> same problem when trying to run postgis tests against a postgresql 9.1
>> database. The protocol for blobs has changed which caused those tests to
>> fail. Since we want to support both major versions of postgresql it makes
>> sense that this version is configurable.
>>
>> Anyways, what you can do is this.
>>
>> 1. Define a property in the root pom, with the default being the current
>> postgresql jdbc driver version.
>>
>> 
>>8.4-701.jdbc3
>> 
>>
>> 2.  Update the dependency management entry.
>>
>>   
>>  postgresql
>>  postgresql
>>  ${postgresql.jdbc.version}
>>
>>
>> 3. In imagemosaic-jdbc/pom.xml simply override the property.
>>
>> 
>>9.1-901-1.jdbc4
>> 
>>
>> -Justin
>>
>> On Fri, Oct 12, 2012 at 2:57 AM,  wrote:
>>
>>> Long story for short. The postgis raster online test was broken and
>>> the fix is to update the postgres driver version. The module  will not
>>> work with 8.x postgres drivers.
>>>
>>> The global pom contains
>>>
>>> 
>>>   postgresql
>>>   postgresql
>>>   8.4-701.jdbc3
>>> 
>>>
>>> The imagemosaic-jdbc pom has to specify its own version.
>>>
>>>   
>>> postgresql
>>> postgresql
>>> 
>>> 9.1-901-1.jdbc4
>>> test
>>>   
>>>
>>> Perhaps we should update the version in the global POM ?.
>>> Btw, our postgis jdbc driver is also not up to date.
>>>
>>> Cheers
>>> Christian
>>>
>>>
>>>
>>> 
>>> This message was sent using IMP, the Internet Messaging Program.
>>>
>>>
>>>
>>>
>>> --
>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> ___
>>> GeoTools-Devel mailing list
>>> GeoTools-Devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>
>>
>>
>>
>> --
>> Justin Deoliveira
>> OpenGeo - http://opengeo.org
>> Enterprise support for open source geospatial.
>>
>
>
>
> 
> This message was sent using IMP, the Internet Messaging Program.
>
>
>
> --
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>

-- 
Ben Caradoc-Davies 
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel


Re: [Geotools-devel] Postgis 2.0 raster and postgres jdbc driver version

2012-10-14 Thread Ben Caradoc-Davies
Justin,

see also this Jira issue from February. If you upgrade the driver and 
fix this failure, please resolve the issue:

Support PostgreSQL 9 hex bytea output format to avoid massive data 
corruption
https://jira.codehaus.org/browse/GEOT-4040

Kind regards,
Ben.

On 12/10/12 21:08, Justin Deoliveira wrote:
> I would use a property to make this configurable. Indeed I ran into this
> same problem when trying to run postgis tests against a postgresql 9.1
> database. The protocol for blobs has changed which caused those tests to
> fail. Since we want to support both major versions of postgresql it
> makes sense that this version is configurable.
>
> Anyways, what you can do is this.
>
> 1. Define a property in the root pom, with the default being the current
> postgresql jdbc driver version.
>
> 
> 8.4-701.jdbc3
> 
>
> 2.  Update the dependency management entry.
>
> 
> postgresql
> postgresql
> ${postgresql.jdbc.version}
> 
>
> 3. In imagemosaic-jdbc/pom.xml simply override the property.
>
> 
> 9.1-901-1.jdbc4
> 
>
> -Justin
>
> On Fri, Oct 12, 2012 at 2:57 AM,  > wrote:
>
> Long story for short. The postgis raster online test was broken and
> the fix is to update the postgres driver version. The module  will not
> work with 8.x postgres drivers.
>
> The global pom contains
>
> 
> postgresql
> postgresql
> 8.4-701.jdbc3
> 
>
> The imagemosaic-jdbc pom has to specify its own version.
>
> 
> postgresql
> postgresql
> 
> 9.1-901-1.jdbc4
> test
> 
>
> Perhaps we should update the version in the global POM ?.
> Btw, our postgis jdbc driver is also not up to date.
>
> Cheers
> Christian
>
>
>
> 
> This message was sent using IMP, the Internet Messaging Program.
>
>
>
> 
> --
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> 
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
>
>
> --
> Justin Deoliveira
> OpenGeo - http://opengeo.org
> Enterprise support for open source geospatial.
>

-- 
Ben Caradoc-Davies 
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel


Re: [Geotools-devel] Postgis 2.0 raster and postgres jdbc driver version

2012-10-14 Thread Andrea Aime
On Fri, Oct 12, 2012 at 5:18 PM, Justin Deoliveira wrote:

> Ahh cool. Good to know.
>
> I will probably still parameterize the version at some point though so we
> can run unit tests against postgres 8 and 9 easily.
>

Sounds like a good idea.
In the past I've tried an upgrade a couple of times, but gave up because
each attempt resulted in
measurably lower performance (not that much of a slowdown, but back at the
time there was no functional
reason to do the upgrade).

Cheers
Andrea

-- 
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it 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

---
--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel


Re: [Geotools-devel] Envelope toString

2012-10-14 Thread Andrea Aime
On Sat, Oct 13, 2012 at 1:48 PM, Jody Garnett wrote:

> It's not about accepting your own pull request, what worries me most is
> changes in the referencing module
> which seem to be directed mostly by the haste of the moment than by a good
> understanding of what
> needs fixing and how.
> Commits are also not directly related to the jiras being opened, for
> example it's hard to tell what
> happened with https://jira.codehaus.org/browse/GEOT-4286, has it been
> reverted?
> One final worry bit is that the changes to referencing do not seem to be
> covered by their own tests
>
> +1. Seems like a faux paux to me to apply these pull requests without
> review from the module maintainer. Especially a module as core as
> referencing.
>
> I applied a pull request to gt-wms, and was waiting for andrea on the one
> involving gt-referencing, since he is busy I backed out those changes and
> made a new pull request.
>
> The "wish" for the change to ft-referencing still stands.
>
> When working in the debugger with the various Envelope classes I would
> like to know what CRS they are working with (even if it is just provided by
> CRS.toSRS method).
>

No problem with that. But I don't understand the change you wanted to make
to CRS,
it seems it contained bits that were backed out too, kind of confusing.


>
> Trying out that idea, shows two test cases are effected (they were
> checking toSting output).
>

Which is a bad idea, the tests should be modified to check the actual
ordinates, not the toString output.

Cheers
Andrea

-- 
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it 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

---
--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel