typeorm icon indicating copy to clipboard operation
typeorm copied to clipboard

feat: implement mongodb transactions

Open nordinh opened this issue 3 years ago • 2 comments

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 master branch
  • [x] npm run format to apply prettier formatting
  • [x] npm run test passes 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?

nordinh avatar Aug 11 '22 11:08 nordinh

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.

pleerock avatar Aug 24 '22 15:08 pleerock

Hi @pleerock ,

Merged master, all tests seem to pass.

Kind regards, Nordin

nordinh avatar Aug 29 '22 13:08 nordinh

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.

pleerock avatar Dec 03 '22 16:12 pleerock

adding @mptr in the case of interest.

pleerock avatar Apr 15 '23 09:04 pleerock

Looks like no community interest here. Closing for now.

pleerock avatar Jan 03 '24 05:01 pleerock