I never quite understood the purpose of TypedFIDMapper but tried to keep the functionality consistent with the changes I made to the FIDMapper classes.
If the DefaultFIDMapperFactory is changed, it will break some of the DB2 junit testcases although this isn't a major problem. Whether it breaks anything else is anyones guess. Regards, David At 02:59 AM 11/8/2006, Andrea Aime wrote: >Justin Deoliveira ha scritto: > > Hi all, > > > > Looking at JDBCDataStore, it seems that when determining a fid mapper, > > the DefaultFIDMapperFactory wraps the fid mapper with a > > "TypedFIDMapper", which prepends the type name to the fid. Andrea, svn > > points to you. > > > > There is a property on the class which controls this behavior. I would > > like to turn off the flag for postgis, but am weary of doing so without > > checking with people first. If need be I can keep the default behavior > > the way it is, and set if via a datastore parameter. > >I do believe the intention of TypedFIDMapper was to ensure the unicity >of the FID among the feature types emitted by the same data store, but >from where I stand I don't remember why I did it. > >Yet, I don't see any objection against its removal... maybe someone >with the jdbc datastore messy architecture more fresh in mind can >point out a reason, but I don't :-) > >Cheers >Andrea > >------------------------------------------------------------------------- >Using Tomcat but need to do more? Need to support web services, security? >Get stuff done quickly with pre-integrated technology to make your job easier >Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo >http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 >_______________________________________________ >Geotools-devel mailing list >Geotools-devel@lists.sourceforge.net >https://lists.sourceforge.net/lists/listinfo/geotools-devel ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel