Alissa Bonas has posted comments on this change.

Change subject: core: Exaple: Use UUID for handling database Guids
......................................................................


Patch Set 1: (2 inline comments)

....................................................
Commit Message
Line 5: CommitDate: 2013-07-01 09:33:41 +0300
Line 6: 
Line 7: core: Exaple: Use UUID for handling database Guids
Line 8: 
Line 9: The Postgres JDBC Driver natively supports the java.lang.UUID type as a
you probably mean java.util.UUID , as there is no java.lang.UUID ... :)
Also, please pay attention that in ovirt-engine project there's a copy of 
java.util.UUID (not clear from which jdk version)
so please check which exact UUID is used by the class loader if you count on 
some specific behavior...
The class is here (as you can see, in wrong package location)
ovirt-engine/frontend/webadmin/modules/gwt-extension/src/main/java/org/ovirt/engine/ui/uioverrides/java/util
Line 10: representation of a database uuid, in both directions -
Line 11: 1. ResultSet#getOtbject(int) and ResultSet#getObject(String) return a
Line 12:    UUID instance when applied to a uuid column.
Line 13: 2. PreparedStatement#setObject(int, Object) can receive a UUID and 
apply


Line 7: core: Exaple: Use UUID for handling database Guids
Line 8: 
Line 9: The Postgres JDBC Driver natively supports the java.lang.UUID type as a
Line 10: representation of a database uuid, in both directions -
Line 11: 1. ResultSet#getOtbject(int) and ResultSet#getObject(String) return a
getOtbject-->getObject
Line 12:    UUID instance when applied to a uuid column.
Line 13: 2. PreparedStatement#setObject(int, Object) can receive a UUID and 
apply
Line 14:    it to a uuid column.
Line 15: 


-- 
To view, visit http://gerrit.ovirt.org/16281
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4d30e1ab7303ecad68dbc9005ac5c6ef733f1707
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Greg Padgett <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Mike Kolesnik <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to