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.

Sign up for GitHub

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

Jump to bottom

AsyncLazyInitializer is not an asynchronous LazyInitializer #413

Closed
sharwell opened this issue Nov 9, 2018 · 2 comments
Closed

AsyncLazyInitializer is not an asynchronous LazyInitializer #413

sharwell opened this issue Nov 9, 2018 · 2 comments
Milestone
v16.0

Comments

@sharwell
Copy link
Member

sharwell commented Nov 9, 2018

Bug description

The name AsyncLazyInitializer can cause confusion for users w.r.t. LazyInitializer. The former is not intended to be an asynchronous form of the latter, and addresses a totally different problem. It should be renamed.
@sharwell sharwell mentioned this issue Nov 9, 2018
Open
@AArnott AArnott added this to the v16.0 milestone Nov 9, 2018
@AArnott
Copy link
Member

AArnott commented Dec 3, 2018

What would you suggest the name be? It's an AsyncLazy<T>-based initializer. So the name is a derivation from AsyncLazy rather than LazyInitializer.

@AArnott
Copy link
Member

AArnott commented Dec 18, 2019

We already shipped this.

@AArnott AArnott closed this as completed Dec 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v16.0
Development

No branches or pull requests
2 participants
@AArnott @sharwell