-
Notifications
You must be signed in to change notification settings - Fork 165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add uuid support #749
Add uuid support #749
Conversation
Fix internal version and variation bits.
…rimary key when creating new EOs.
}, | ||
{ | ||
columnName = uuid; | ||
externalType = bytea; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Samuel,
Would it be possible to use PostgreSQL's uuid
as the external type here? Or does it create problems elsewhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Paul,
PG UUID seems to map to Java UUID and is a very PG specific. I do not remember having tried to map a UUID column to an NSData with EOF.
Keep in mind that EOF only supports Integer, Long, String, Date and NSData as primary key type.
Samuel
Le 28 juin 2016 à 07:19, Paul Hoadley [email protected] a écrit :
In Frameworks/Core/ERPrototypes/Resources/erprototypes.eomodeld/EOJDBCPostgresqlPrototypes.plist #749 (comment):
@@ -359,6 +359,13 @@
valueClassName = NSString;
valueType = S;
width = 10000000;
},
{
columnName = uuid;
Hi Samuel,externalType = bytea;
Would it be possible to use PostgreSQL's uuid https://www.postgresql.org/docs/current/static/datatype-uuid.html as the external type here? Or does it create problems elsewhere?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/wocommunity/wonder/pull/749/files/66093a103c504f7328258ea4c5a59052244c0d08..bde27aea1cebb6979543b5bedd9e0a78894dba38#r68738897, or mute the thread https://github.com/notifications/unsubscribe/AA-cjlxQd5Iz-penZKgBNvWmacj57584ks5qQQNXgaJpZM4In8EK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, thanks Samuel—it was lost on me that your overall aim was primary key support.
Add support for auto generated UUID type 4 (random) as primary key.
Add a uuid prototype for FrontBase, MySQL and PostgreSQL. I do not have instance of other type of database to test them.
Add support for UUID primary key in ERRest.