-
Notifications
You must be signed in to change notification settings - Fork 199
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 MongoDB + skip by dialect + environment variables for MongoDB #77
Conversation
…e mapping for Calendar and Byte for BuiltInTypeTest to pass
… MongoDBTupleSnapshot. fix endsWith check on _id to include . prefix. add ToString on AssociationTupleSnapshot. construct DBObject with size instead of HashMap for columnNames+columnValues.
…_, eg. associations_BlogEntry
…ng MongoDB instance
By the way the reason the previous code was not working is because in my system, Environment from ORM reads System properties before the the TestHelper was initialized. I am not sure why but after all nothing is guaranteed wrt class loading and static block calls ordering. |
Ok my bad! But not fully understanding it. I know there are no guarantees among static class initialization, but there definitely is an order between a) initialize the class b) use it. |
I'm working on merging this one to master :) |
thanks! |
I've reworked @Sanne environment variables for MongoDB commit.
It should properly work now. Can you review @Sanne ?
The rest is good and you have the +1 from me to apply.