Skip to content

impr: optimize database access for UserDal (@fehmer)#5544

Merged
Miodec merged 2 commits intomasterfrom
feature/optimize-user-dal
Jul 1, 2024
Merged

impr: optimize database access for UserDal (@fehmer)#5544
Miodec merged 2 commits intomasterfrom
feature/optimize-user-dal

Conversation

@fehmer
Copy link
Copy Markdown
Member

@fehmer fehmer commented Jun 28, 2024

Optimize database access for UserDal:

  • Only fetch a subset of the document using projection if a small subset of the user document is needed
  • Detect missing user by inspecting the updateOne result instead of fetching the user first

@monkeytypegeorge monkeytypegeorge added the backend Server stuff label Jun 28, 2024
Miodec
Miodec approved these changes Jul 1, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@Miodec Miodec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@Miodec Miodec merged commit d566ba6 into master Jul 1, 2024
@Miodec Miodec deleted the feature/optimize-user-dal branch July 1, 2024 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Miodec Miodec Miodec approved these changes

Assignees

No one assigned

Labels

backend Server stuff

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fehmer @Miodec @monkeytypegeorge