Hi David, I was actually going to leave DefautlFIDMapperFactory alone, and only make the change in PostgisFIDMapperFactory, so all other jdbc modules should remain unaffected.
-Justin David Adler wrote: > 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 >> [email protected] >> 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 > [email protected] > https://lists.sourceforge.net/lists/listinfo/geotools-devel > > !DSPAM:1004,4551c7f5208181775926497! > -- Justin Deoliveira The Open Planning Project [EMAIL PROTECTED] ------------------------------------------------------------------------- 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 [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
