Skip to content
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

[Discussion] Unit of Work Accumulation #1

Open
ryankauk opened this issue Jan 11, 2021 · 2 comments
Open

[Discussion] Unit of Work Accumulation #1

ryankauk opened this issue Jan 11, 2021 · 2 comments

Comments

@ryankauk
Copy link

ryankauk commented Jan 11, 2021

Hey there, I wanted to get your thoughts on the unit of work during requests with your current implementation. As it it right now, if you fetched your api once mikro would go to the database to get the object and then store it in it's unit of work. If you fetch the second time it will grab from it's unit of work. Depending on what you're doing on the handler and if it has authentication the unit of work will become unreliable in these scenarios. Have you ran into this issue?

I ended up wrapping all my logic inside the RequestContent.createAsync but this would have to be done on every endpoint. Was just wondering your thoughts on this.

@tolgap
Copy link

tolgap commented Sep 29, 2021

I agree, this repo is a very optmistic way of doing things. Starting your ORM inside an API route handler? That will scale very poorly.

@jonahallibone
Copy link
Owner

Hey guys -- this repo was created originally when I intended on using mikro in a blueprint for future projects. It was also intended to be consumed in a serverless environment which didn't seem to have a usable solution without creating too many connections to the DB without a proxy. This repo worked for my uses at the time, but has become outdated and is not maintained anymore. If you have a better way of handling mikro in Next for serverless, feel free to fork and PR, or ask the author of the mikro repo to replace my example with yours.

jonahallibone pushed a commit that referenced this issue Nov 22, 2021
…0.3.2 (#1)

Bumps [babel-plugin-transform-typescript-metadata](https://github.com/leonardfactory/babel-plugin-transform-typescript-metadata) from 0.3.0 to 0.3.2.
- [Release notes](https://github.com/leonardfactory/babel-plugin-transform-typescript-metadata/releases)
- [Changelog](https://github.com/leonardfactory/babel-plugin-transform-typescript-metadata/blob/master/CHANGELOG.md)
- [Commits](leonardfactory/babel-plugin-transform-typescript-metadata@v0.3.0...v0.3.2)

---
updated-dependencies:
- dependency-name: babel-plugin-transform-typescript-metadata
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants