search

synopse / mORMot

Synopse mORMot 1 ORM/SOA/MVC framework - Please upgrade to mORMot 2 !
https://synopse.info
788 stars 325 forks source link

Update SynMongoDB.pas for using all connections of a cluster #363

Closed danielkuettner closed 4 years ago

danielkuettner commented 4 years ago

Hi Arnaud, with these changes (I've pasted all manually over Web, hope there are not too much mistakes) alle Connections of a Client Cluster could be used. The rpNearest is a new constant for Mongo 4.4 and I wanted to intruduce it for further using. Atm it should be a load balancing of using all Connections evenly. This could be usefull when many client are connecting to a Atlas Cluster in case of readings. If you have questions, Email me.

Daniel

danielkuettner commented 4 years ago

Thanks!

Am 02.11.2020 um 15:17 schrieb ab notifications@github.com:

@synopse approved this pull request.

It sounds fine. I will just make small refactoring. Thanks!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/synopse/mORMot/pull/363#pullrequestreview-521666254, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQG4PM2OZRXVN6ETCN52CGDSN25QHANCNFSM4TGUAXQA.

danielkuettner commented 4 years ago

Pull request #362 is also necessary to work with slave connections.

Am 02.11.2020 um 15:17 schrieb ab notifications@github.com:

Merged #363 https://github.com/synopse/mORMot/pull/363 into master.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/synopse/mORMot/pull/363#event-3948318086, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQG4PM62VTSMBTLMPASQHGTSN25QZANCNFSM4TGUAXQA.

synopse commented 4 years ago

Please check if my review is fine. I changed the "nearest" implementation somewhat.

danielkuettner commented 4 years ago

I’ve test it and it worked! Your implementation of nearest is a optimized version of what I’ve wanted.

Thanks and hopefully we will meet us in the future! Daniel

Am 02.11.2020 um 15:51 schrieb ab notifications@github.com:

Please check if my review is fine. I changed the "nearest" implementation somewhat.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/synopse/mORMot/pull/363#issuecomment-720518836, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQG4PM35N3HFIHPNG5HNLFDSN3BPTANCNFSM4TGUAXQA.