From: John H. <Joh...@li...> - 2011-11-16 03:20:36
|
There is: modules/library/jdbc/src/test/java/org/geotools/jdbc/JDBCUuidTest.java tho it only tests getting features. I will fill it out a bit more. From: Jody Garnett [mailto:jod...@gm...] Sent: Wednesday, 16 November 2011 1:48 PM To: John Hudson Cc: Geo...@li... Subject: Re: [Geotools-devel] UUID Support I suspect John that we need to write some test cases to confirm the functionality works (and remains working). Did the patch include a JUnit test case? If not could you write one and attach it to the JIRA? Thanks muchly, Jody Garnett -- Jody Garnett On Wednesday, 16 November 2011 at 2:08 PM, John Hudson wrote: Hello, I have been working on getting a datastore with a primary key as a UUID and found this: http://jira.codehaus.org/browse/GEOT-3016 I have applied it to the current trunk and have done some tests which show its working. I noted the TODO’s on the JIRA: 1. Implement the UUID support into PostGisPsDialect 2. UUID insert/update 3. UUID as primary key From my testing: 1. PostGisPsDialect seems to be a pass-through to the PostGISDialect.java so should be left as is? 2. insert and update seem to be working 3. my lack of knowledge in this area means I’m not sure what this one really means Is there a chance this patch can be reviewed and applied to the trunk ? If more needs doing I’m happy to more this forward. *a note about the patch – its double archived, to get to the actual patch you must unzip the file in the .gz file. – which is a actually a .patch extension. My lack of understanding might have pointed me in the entirely wrong direction here, any pointers a definitely welcome. Thanks for your time, John Hudson ________________________________ The contents of this email are confidential and may be subject to legal or professional privilege and copyright. No representation is made that this email is free of viruses or other defects. If you have received this communication in error, you may not copy or distribute any part of it or otherwise disclose its contents to anyone. Please advise the sender of your incorrect receipt of this correspondence. ------------------------------------------------------------------------------ RSA(R) Conference 2012 Save $700 by Nov 18 Register now http://p.sf.net/sfu/rsa-sfdev2dev1 _______________________________________________ Geotools-devel mailing list Geo...@li...<mailto:Geo...@li...> https://lists.sourceforge.net/lists/listinfo/geotools-devel ________________________________ The contents of this email are confidential and may be subject to legal or professional privilege and copyright. No representation is made that this email is free of viruses or other defects. If you have received this communication in error, you may not copy or distribute any part of it or otherwise disclose its contents to anyone. Please advise the sender of your incorrect receipt of this correspondence. |