Multi-async in Entity Framework 6?

Question

This is my code:

var banner = context.Banners.ToListAsync()
var newsGroup = context.NewsGroups.ToListAsync()
await Task.WhenAll(banner, newsGroup);

But when i called the function from controller. It showed error

A second operation started on this context before a previous asynchronous operation completed. Use 'await' to ensure that any asynchronous operations have completed before calling another method on this context. Any instance members are not guaranteed to be thread safe.

Please help me solve this issue.

I have 2 task. If i run each task. it's success. but if i run like my code above. It's error — An Hv, Dec 17 '13 at 08:16

score 121 · Accepted Answer · answered Dec 17 '13 at 13:10

121

The exception explains clearly that there is only one asynchronous operation per context allowed at a time.

So, you either have to await them one at a time as the error message suggests:

var banner = await context.Banners.ToListAsync();
var newsGroup = await context.NewsGroups.ToListAsync();

Or you can use multiple contexts:

var banner = context1.Banners.ToListAsync();
var newsGroup = context2.NewsGroups.ToListAsync();
await Task.WhenAll(banner, newsGroup);

answered Dec 17 '13 at 13:10

Stephen Cleary

437,863
77
675
810

35

just a note, if you have a Lazy variable that uses the context in the query even with the await it will throw the same error, just get the property before the query, it was a pain to find out this. – Pedro.The.Kid May 28 '14 at 15:58
7

@Pedro.The.Kid: As a general rule, don't use lazy loading with asynchronous DB access. Lazy loading is always synchronous, so it's far better to use Include or separate queries for the additional data. – Stephen Cleary May 28 '14 at 16:13
1

Is there any specific reason why you need a context per async query? I feel like this becomes quite a bit of a limiting factor. – Zapnologica Jan 17 '16 at 08:17
1

@Zapnologica: It's just the way ES6 was designed. Each context can only handle one query *at a time*. So if you finish one query before the next one begins, you'd only need one context. It's only a problem if you want to do multiple queries at the same time. – Stephen Cleary Jan 17 '16 at 12:55
@StephenCleary, I am having a hard time finding that query as I there isn't anything immediately before the exception. Is there a way for us to find what is currently being executed? Thanks – Fabio Milheiro May 17 '16 at 15:17
BTW, if I debug, it works! Which means the other thing just happens to have enough time to execute. – Fabio Milheiro May 17 '16 at 15:18
@Bomboca: Not that I know of. – Stephen Cleary May 17 '16 at 17:26
Got it, I just had to look at what lines executed before even though they were not in the same file but not that hard. Thanks for answering. – Fabio Milheiro May 17 '16 at 19:23
Thank You, it works. I have a method call which has two async calls. I had to put await before both of them, and only then did this exception stopped. – VivekDev Jan 03 '17 at 02:43
But if you cant let them run simultaneously by putting await before each one , is there any significant gain from the async code? – Ernesto Aug 08 '17 at 15:38
@StephenCleary unable to run multiple queries in parallel with the one db context. Is it just me or is that disappointing? – Andrew Duffy Dec 21 '18 at 02:48
@AndrewDuffy: It's not just you. :) – Stephen Cleary Dec 21 '18 at 17:55
I am facing the same issue even adding the await while performing db operation. In my scenario I have multiple DbContext and I am spawning 1000 Task using ParallelForEachAsync. Initially I had a single DbContext ,later I have added separate DbContext for two different repository. @StephenCleary your input on my scenario. – Jaydeep Shil Oct 12 '20 at 09:26
@JaydeepShil: If you have 1000 parallel tasks and each of them is doing db work, then you will need 1000 DbContexts for each db. – Stephen Cleary Oct 12 '20 at 12:56
Thank you @StephenCleary for your reply. What should be my design approach here ? I want to spawn 1000 or it can be more items on parallel tasks , each of the task will be performing a read and after that update and then printing those documents in printer. How can I make my dbcontext threadsafe for all parallel operations ? as you also mentioned 1000 DbContext would be required but I have a singleton dbcontext which is injected and this will be a single endpoint call and perform the parallel task in a async scheduler. – Jaydeep Shil Oct 12 '20 at 13:25
@JaydeepShil: You'll need a separate DbContext for each one. – Stephen Cleary Oct 12 '20 at 14:04
@StephenCleary able to solve the issue by adding asynclock before performing async dbcontext operation using SemaphoreSlim. before async get/update I am doing await semaphore.WaitAsync().ConfigureAwait(false); and do the awaited operation and release the semaphore. – Jaydeep Shil Oct 13 '20 at 06:26
@JaydeepShil: OK, but then what's the point of 1000 tasks if they're no longer concurrent? – Stephen Cleary Oct 13 '20 at 13:34
@StephenCleary as dbContext is not thread safe we cant make it concurrent , but to make it concurrent we have to create a new dbcontext for each Task operation , currently our dbcontext is already injected and we are using the injection call from repository , to create a new dbcontext object there will be lot of code changes as there are many flow. – Jaydeep Shil Oct 14 '20 at 07:18
@JaydeepShil: I'm not sure what you want me to say? There's no magic answer that makes this easy. If you want to change from `1` database operation to `n` concurrent database operations, and if your code uses DbContext, then you will need to use `n` DbContexts. That will require changes in how your DbContexts are constructed - i.e., a DbContextFactory or just creating new ones as necessary. – Stephen Cleary Oct 14 '20 at 17:22

score 4 · Answer 2 · edited Sep 08 '17 at 05:55

4

If you are using IoC container for your Data Provider injection, consider to use "transient" or "PerWebRequest" type for your lifecycle.

For example: https://github.com/castleproject/Windsor/blob/master/docs/lifestyles.md

edited Sep 08 '17 at 05:55

Vitaliy Ulantikov

10,157
3
61
54

answered Nov 11 '16 at 20:38

Teoman shipahi

47,454
15
134
158

score 3 · Answer 3 · answered Jul 18 '17 at 09:30

If you use Unity for dependency injection with for example repository pattern you will get the following error using two or more contexts with create/update/delete:

The relationship between the two objects cannot be defined because they are attached to different ObjectContext objects.

This can be solved using PerRequestLifetimeManager. More info here:

C# EF6 make multiple async calls to one context using Unity - Asp.Net Web Api

container.RegisterType<DbContext>(new PerRequestLifetimeManager());
container.RegisterType<ISupplierRepository, SupplierRepository>();
container.RegisterType<IContactRepository, ContactRepository>();

Multi-async in Entity Framework 6?

3 Answers3

Linked