Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [udig-devel] generic geotools datastore support

On 23/04/2010, at 10:01 PM, Ugo Taddei wrote:

> Jody Garnett wrote:
>> Yeah progress! Well progress bar; well not say yeah :-P
> 
> Progress yes, but progress bar, no. Grey dialogs.
> 
>> I am not sure what we should do about progress calculating bounding boxes :-(
>> I have thought of popping up a dialog and asking a user for the bounds if it "takes too long"; but if you have many many tables that will be a trouble. I could try putting off the calculation until you first use the layer ...
> 
> I wonder why udig is trying to get the bbox at this point. It should first list the tables, and only get the bbox after I select the tables. But this is probably something coming from geotools.

Bit of both.

uDig want to know the bounds now - so it can cache for later searching. When you search using the search view you have the option to search within the bounds of the current screen. 

> FYI: the dialog is still blocked. I'm waiting to see if it'll return before I start my weekend :-)

And this is the part that is in geotools; we are dependent on the implementation provided to return us a good bounds - and many implementations default to scanning the entire table :-(

> I also get loads of
> 
> WARNING: No primary key or unique index found for MY_TABLE.
> Apr 23, 2010 11:13:59 AM org.geotools.jdbc.JDBCDataStore getPrimaryKey
> 
> But I think/hope this is only a warning.

I think so - it tries to figure out a good "FeatureID" using any primary keys.

Jody

Back to the top