feat: implement mongodb transactions
This new feature adds transaction support for MongoDB
Closes: #3051
Description of change
Adding support for transactions in MongoDB (transactions only work on replicasets). By default, transactions are not enabled as is currently the default. Transactions can be enabled through connection options.
Pull-Request Checklist
- [x] Code is up-to-date with the
masterbranch - [x]
npm run formatto apply prettier formatting - [x]
npm run testpasses with this change - [x] This pull request links relevant issues as
Fixes #0000 - [ ] There are new or updated unit tests validating the change
- [x] Documentation has been updated to reflect this change
- [x] The new commits follow conventions explained in [CONTRIBUTING.md]
Remark
Not added any new tests for this, as it requires quite a different MongoDB setup (replicaset/cluster). Adding such a cluster in thee default docker-compose would make that a lot heavier. Any suggestions on how to proceed?
Not added any new tests for this, as it requires quite a different MongoDB setup (replicaset/cluster). Adding such a cluster in thee default docker-compose would make that a lot heavier. Any suggestions on how to proceed?
Not sure about this one. We can skip tests for now.
Also, please merge latest master into your branch, since it contains fixes to test cases, to make sure all tests are running on your branch.
Also I need to review from people who use MongoDB before we can merge this PR.
Thanks for contribution.
Hi @pleerock ,
Merged master, all tests seem to pass.
Kind regards, Nordin
I cannot really review mongodb features, because it's community-lead driver and I personally don't work a lot with mongodb. I need community help to review mongodb-related PRs. There are many people contributing to mongodb driver, but none of them willing to review other people's mongodb PRs.
adding @mptr in the case of interest.
Looks like no community interest here. Closing for now.