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

Further increase performance of DIGraph in production by not needing override feature #188

Open
levibostian opened this issue Jun 3, 2024 · 0 comments

Comments

@levibostian
Copy link
Owner

The DIGraph is auto-generated. The graph that's generated today is great except 1 thing I would like to improve.

The production and testing graph are identical. The graph contains this feature that allows you to override an instance in the graph which is used to provide a mock for testing. This override feature has a performance hit because it uses a mutex. Ideally, you don't need this feature in production, just testing. It would be great if the production graph didn't include this feature to make it even more performant at runtime.

Be sure that whatever solution is chosen does not introduce side effects into the test suite. The test suite should continue to be easy to use.

@levibostian levibostian converted this from a draft issue Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog/Refinement
Development

No branches or pull requests

1 participant