search

appscot / sails-orientdb

OrientDB adapter for Waterline / Sails.js ORM
MIT License
25 stars 23 forks source link

Support 'LONG' type #142

Open nkibler7 opened 8 years ago

nkibler7 commented 8 years ago

From the sounds of it, Waterline wants this to be at the database adapter level, per this issue. This has already been done to sails-mysql as well.

OrientDB does have long type support (-263 through 263-1) and it would be nice to use it, having it persist through schema migrations. For now, I can change my OrientDB field type directly, but if I perform a schema migration, I'll have to change it again.

sixpounder commented 8 years ago

I suggest you to follow this thread https://github.com/balderdashy/waterline/pull/981. In the meanwhile, you could edit the orientdb adapter in a way similar to the one I used in the PR you referenced, I think there should be no problems with that (if tests are ok)