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

Per request configuration store #224

Closed
mikz opened this issue Jan 17, 2017 · 0 comments · Fixed by #289
Closed

Per request configuration store #224

mikz opened this issue Jan 17, 2017 · 0 comments · Fixed by #289
Assignees

Comments

@mikz
Copy link
Contributor

mikz commented Jan 17, 2017

Configuration Store instance is now used globally in the provider module.
That is unfortunate for several reasons, but main of them is multi-tenancy.

There should be a way to have per request configuration store instance, so it can't be shared with any other request. That way there would be no need to handle configuration expiration, memory leaks and invalidation at lest when configuration reloading is enabled.

@mikz mikz self-assigned this Jan 17, 2017
@mikz mikz added this to the On-premise ER5 release milestone Mar 7, 2017
@mikz mikz added the B-Next label Mar 7, 2017
@ghost ghost added B-current and removed B-Next labels Mar 9, 2017
@mikz mikz closed this as completed in #289 Mar 9, 2017
@ghost ghost removed the B-current label Mar 9, 2017
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

Successfully merging a pull request may close this issue.

1 participant