Hello.  I have some questions to ask.

When  the feature is  being added to the non-AUTO_COMMIT transaction using
AbstractFeatureStore (has been got from ShapefileDataStore) - before to add
this feature into the TransactionStateDiff the DiffFeatureWriter (one of the
writers in the delegate sequence)  reads all the existing features ( ) and
then adds the passed features to the TransactionStateDiff.

If there are 10000 features in the shapefile (usual case)  and heavy editing
operations are performed over the underlying datastore - each usage of
methods through FeatureStore leads to reading all the features..

Performance strongly suffers..

I use TransactionStateDiff's internal Map of differences directly while
putting modified/new/removed feature to the map with its FIDs as a keys..
works very well and quickly without that reading all the features each
time.. (that is a really case when there are thousands of records in SHPs).
Then it is just needed to commit transaction and all the features are
persisted. 

So from that point the question appears.. why AbstractFeatureStore is so
non-optimized for modifications.

One answer might be integrity of the datastore (for example, checking that
modified feature exists with specified FID, etc..). But when the
functionality is to read some features from the datastore, change some
attributes without changing FIDs and put them back to the transaction - this
approach that is used with reading and checking (inside of
AbstractFeatureStore) is slow and unoptimized.

Possibly I missed some important points, correct me if that's so.


Vitali.




-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to