Skip to content
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.

Bind to legacy driver #11

Closed
wants to merge 5 commits into from
Closed

Bind to legacy driver #11

wants to merge 5 commits into from

Conversation

vegar
Copy link

@vegar vegar commented Aug 27, 2015

While we wait for the new mongo c# driver to get LINQ support, I have compiled MongoRepository against the legacy driver.

The reason for this, is to be able to use both the new 2.0 api and MongoRepository in the same project with out getting into conflicts on mongo driver version.

@RobThree
Copy link
Owner

See my comment here.

@vegar
Copy link
Author

vegar commented Aug 27, 2015

All your comments are perfectly valid.

We are in a situation where our project have gotten a little too much 'MongoRepository'-juice all over, and are looking for a way to move forward. Getting access to the new 2.0 aggregation-framework would help us much, but as it is, MongoRepository is kind of blocking the way. I'm in no way blaming you or MongoRepository for that. It gave us a great head-start early in the project, but we failed at isolating our data access from the rest of the application.

Anyway - this pull-request is sort of a 'do whatever takes us forward' with a heavy weight on us...

I'll look into the deprecated methods and replace the implementation where an alternative exist.

For the net3.5/4.0.... As I understand, Mongo have left <4.5 behind.

@vegar
Copy link
Author

vegar commented Aug 27, 2015

I need to check that the new implementations of GetTotalDataSize() and GetTotalStorageSize() returns the same values as before.

The RequestStart and RequestDone methods are still there, but are made internal. I'll need to spend some more time to see what the alternative would be, and why it was hidden in the first place.

@RobThree
Copy link
Owner

See #4

@RobThree RobThree closed this Dec 18, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants