Re: [Geotools-devel] Confusion looking at the code in JDBCDataStore

2013-06-12 Thread Christian Mueller
Ok Justin, I will fix the bug, continue investigation and send you a report.

Thanks
Christian


2013/6/12 Justin Deoliveira 

>
>
>
> On Wed, Jun 12, 2013 at 8:15 AM, Christian Mueller <
> christian.muel...@os-solutions.at> wrote:
>
>> 1)
>>
>> The method
>>
>> buildEnvelopeAggregates
>>
>> causes some confusion for me.
>>
>> I have a sql table with 3 Point attributes, each with a different SRSid.
>>
>> The method iterates over all geometry attributes for extent calculation,
>> but
>> the sql attribute name passed to the dialect is always the attribute of
>> the default geometry.
>>
>> I assume this is a bug ?
>>
>> Hmmm... agreed, this does look like a bug.
>
>
>> 2)
>>
>> Even if 1)  would work in the correct manner, all the extents are merged
>> resulting in one
>> single extent (method getBounds).  This is correct if all geometry
>> attributes have the same
>> SRSid, but it is nonsense if these attributes do not have the same SRSid.
>>
>> Does this mean that geometry attributes with different SRSids are not
>> supported ?
>>
>> I don't think so. If you look at the code that processes the bounds query
> it processes each envelope returned by the dialect and checks for an
> instanceof ReferencedEnvelope and merges it in taking the crs into account.
> There might be a bug there but at least supporting difference crs's is
> intended.
>
> Thanks for your help
>>
>> Christian
>>
>>
>>
>>
>>
>>
>> --
>> DI Christian Mueller MSc (GIS), MSc (IT-Security)
>> OSS Open Source Solutions GmbH
>>
>>
>>
>> --
>> This SF.net email is sponsored by Windows:
>>
>> Build for Windows Store.
>>
>> http://p.sf.net/sfu/windows-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.
>



-- 
DI Christian Mueller MSc (GIS), MSc (IT-Security)
OSS Open Source Solutions GmbH
--
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel


Re: [Geotools-devel] Confusion looking at the code in JDBCDataStore

2013-06-12 Thread Justin Deoliveira
On Wed, Jun 12, 2013 at 8:15 AM, Christian Mueller <
christian.muel...@os-solutions.at> wrote:

> 1)
>
> The method
>
> buildEnvelopeAggregates
>
> causes some confusion for me.
>
> I have a sql table with 3 Point attributes, each with a different SRSid.
>
> The method iterates over all geometry attributes for extent calculation,
> but
> the sql attribute name passed to the dialect is always the attribute of
> the default geometry.
>
> I assume this is a bug ?
>
> Hmmm... agreed, this does look like a bug.


> 2)
>
> Even if 1)  would work in the correct manner, all the extents are merged
> resulting in one
> single extent (method getBounds).  This is correct if all geometry
> attributes have the same
> SRSid, but it is nonsense if these attributes do not have the same SRSid.
>
> Does this mean that geometry attributes with different SRSids are not
> supported ?
>
> I don't think so. If you look at the code that processes the bounds query
it processes each envelope returned by the dialect and checks for an
instanceof ReferencedEnvelope and merges it in taking the crs into account.
There might be a bug there but at least supporting difference crs's is
intended.

Thanks for your help
>
> Christian
>
>
>
>
>
>
> --
> DI Christian Mueller MSc (GIS), MSc (IT-Security)
> OSS Open Source Solutions GmbH
>
>
>
> --
> This SF.net email is sponsored by Windows:
>
> Build for Windows Store.
>
> http://p.sf.net/sfu/windows-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 SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel


[Geotools-devel] Confusion looking at the code in JDBCDataStore

2013-06-12 Thread Christian Mueller
1)

The method

buildEnvelopeAggregates

causes some confusion for me.

I have a sql table with 3 Point attributes, each with a different SRSid.

The method iterates over all geometry attributes for extent calculation, but
the sql attribute name passed to the dialect is always the attribute of the
default geometry.

I assume this is a bug ?

2)

Even if 1)  would work in the correct manner, all the extents are merged
resulting in one
single extent (method getBounds).  This is correct if all geometry
attributes have the same
SRSid, but it is nonsense if these attributes do not have the same SRSid.

Does this mean that geometry attributes with different SRSids are not
supported ?

Thanks for your help

Christian






-- 
DI Christian Mueller MSc (GIS), MSc (IT-Security)
OSS Open Source Solutions GmbH
--
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel