51

When my table is updated by another party, the db context in dotnet core still return the old value, how can I force the Db context to refresh?

I've done research but I only found people use Reload method, which is not available in EF core, to force the context to refresh.

Some other solution suggests dispose the context after using, but I get error saying the DB context is created by dependency injection and I should not mess up with it.

Camilo Terevinto
  • 31,141
  • 6
  • 88
  • 120
MiDaa
  • 1,149
  • 1
  • 11
  • 25
  • They are sharing the same context? After Saving Changes different context should share the same state. – Fals Sep 13 '17 at 19:10
  • @Fals Unfortunately they do not share the same context. – MiDaa Sep 15 '17 at 09:31
  • 1
    if you only need to read data then "AsNoTracking" could help you out. EF Core: https://learn.microsoft.com/en-us/ef/core/querying/tracking and this for more information: EF Cache Busting: http://codethug.com/2016/02/19/Entity-Framework-Cache-Busting/ – A.J.Bauer Oct 24 '17 at 08:47

8 Answers8

62

Oh, this issue had me in knots for days.

I'm using Visual Studio 2017 with .Net Core 2.1, and my EF Core code looked something like this:

//  1.  Load a [User] record from our database 
int chosenUserID = 12345;
User usr = dbContext.Users.FirstOrDefault(s => s.UserID == chosenUserID);

//  2. Call a web service, which updates that [User] record
HttpClient client = new HttpClient()
await client.PostAsync("http://someUrl", someContent);

//  3. Attempt to load an updated copy of the [User] record
User updatedUser = dbContext.Users.FirstOrDefault(s => s.UserID == chosenUserID);

At step 3, it would simply set "updatedUser" to the original version of the [User] record, rather than attempting to load in a fresh copy. So, if, after step 3, I modified that [User] record, I'd actually lose whatever settings the web service had applied to it.

I - eventually - found two solutions.

I could change the ChangeTracker settings. This worked, but I was concerned about the side-effects of doing this:

dbContext.ChangeTracker.QueryTrackingBehavior = Microsoft.EntityFrameworkCore.QueryTrackingBehavior.NoTracking;

Or, I could slip in the following command, before attempting to reload the [User] record...

await dbContext.Entry(usr).ReloadAsync();

This seems to force .Net Core to reload that [User] record, and life is good again.

I hope this is useful...

Tracking down, and fixing this bug took me days....

There's also an excellent article describing the various ways to get around this caching issue here.

Mike Gledhill
  • 27,846
  • 7
  • 149
  • 159
  • 2
    I understand how frastrated it is. However here’s my suggestion and my current understanding, in the design of EF, the DbContext object is a implementation of unit of work pattern. It should be short lived, and not to be used by different translation. You can try to use another new DbContext to do the second operation. :) – MiDaa Nov 02 '18 at 14:22
  • 2
    Almost a year late but with this solution using Reload, I encounter another error which is "A second operation started on this context before a previous operation completed. This is usually caused by different threads using the same instance of DbContext, however instance members are not guaranteed to be thread safe. This could also be caused by a nested query being evaluated on the client, if this is the case rewrite the query avoiding nested invocations." due to subsequent calls. Probably the ChangeTracker one is better in this case but I am not sure about the side effect, too. – Shinigamae Jan 08 '20 at 09:09
  • Quoting from https://learn.microsoft.com/en-us/ef/core/querying/tracking#tracking-queries : _When the results are returned in a tracking query, EF Core will check if the entity is already in the context. If EF Core finds an existing entity, then the same instance is returned. EF Core won't overwrite current and original values of the entity's properties in the entry with the database values._ So this shows why EF Core does not update an entity when querying it again from the database, which is weird imho – Xriuk Feb 23 '23 at 10:03
  • @Shinigamae the side effects of disabling querytracking at runtime is that the following queries won't be tracked, but all the previously tracked entities should remain in the context. It's the same as calling `AsNoTracking()` at the end of the query. In every case I still recommend to use `NoTrackingWithIdentityResolution` instead because the above would return different instances for the same entity in a single query while this option would consolidate them first – Xriuk Feb 23 '23 at 10:09
38

Dependency Injection and DbContext

You mention that when you try to recreate your DbContext, you get an error about the context being managed by your dependency injection (DI) system. There are two different styles of using a dependency injection system for object creation. The DI can either create a global singleton instance that is shared as a service between all consumers or it can create an instance per scope/unit of work (e.g., per request in a web server).

If your DI system is configured to create a single global shared instance of DbContext, then you will encounter various problems associated with long-lived DbContext.

  • DbContext, by design, never automatically removes objects from its cache because it is not designed to be long-lived. Thus, a long-lived DbContext will retain memory wastefully.
  • Your code will never see changes to items loaded into its cache without manually reloading each entity it loads.
  • DbContext only allows one query to run at any time and is not threadsafe. If you try to run multiple queries on a globally shared instance, it will throw DbConcurrencyException (at least on its async interface, not sure about its sync interface).

Thus, the best practice is to use a single DbContext per unit of work. Your DI system can help you with this by being configured to provide a fresh instance for each request your application processes within a scope. For example, ASP.NET Core’s Dependency Injection system supports scoping instances by request.

Refreshing a Single Entity

The easiest way to get fresh data is to create a new DbContext. However, within your unit of work, or within the constraints of the granularity of scoping provided by your DI system, you may trigger an external process which is supposed to modify your entity directly in the database. You may need to see that change before exiting your DI’s scope or completing your unit of work. In that case, you can force a reload by detaching your instance of the data object.

To do this, first get the EntityEntry<> for your object. This is an object which lets you manipulate DbContext’s internal cache for that object. You can then mark this entry detached by assigning EntitytState.Detached to its State property. I believe that this leaves the entry in the cache but causes the DbContext to remove and replace it when you actually load the entry in the future. What matters is that it causes a future load to return a freshly loaded entity instance to your code. For example:

var thing = context.Things.Find(id);
if (thing.ShouldBeSentToService) {
    TriggerExternalServiceAndWait(id);

    // Detach the object to remove it from context’s cache.
    context.Entities(thing).State = EntityState.Detached;

    // Then load it. We will get a new object with data
    // freshly loaded from the database.
    thing = context.Things.Find(id);
}
UseSomeOtherData(thing.DataWhichWasUpdated);
binki
  • 7,754
  • 5
  • 64
  • 110
  • 2
    Nice answer, I had used a short live context to solve the problem in that project. Thanks and hopefully this answer will help other people. – MiDaa Jul 15 '18 at 10:19
  • 4
    Also worth noting that when you register a DbContext it also registers `DbContextOptions` where `T` is your context type. So you can actually inject that instead (eg. `DbContextOptions rrContextOptions`) and just create a brand new context as needed (`using (var db = new RRContext(rrContextOptions))`. You'll have to make a judgement whether this is best for compared to other answers, but sometimes it's the easiest way since they don't expose the reset functionality publically (it does exist and is used by the DbContext pooling feature). – Simon_Weaver Feb 03 '19 at 22:03
  • 1
    Just for everyone getting here later on, the EntitityFrameworkCore change tracker allows reloading of a single entity: Entities(thing).Reload(); – XIU Jul 24 '21 at 14:36
  • 1
    So this half way helped me in EF 5.0.11. i did have to .Detached, then (item.SubItemInAnotherTable).ReloadAsync(), then ask for the item again with .FirstOrDefault() – Omzig Oct 15 '21 at 20:25
  • Correction, i had to remove ReloadAsync, it was causing a db error like this: 'System.InvalidOperationException: The instance of entity type 'DomainUser' cannot be tracked because another instance with the same key value for {'Id'} is already being tracked. When attaching existing entities, ensure that only one entity instance with a given key value is attached. Consider using 'DbContextOptionsBuilder.EnableSensitiveDataLogging' to see the conflicting key values.' – Omzig Oct 15 '21 at 20:55
  • 1
    @Omzig So you’re confirming that my answer works as written? ;-) – binki Oct 16 '21 at 01:13
  • About detaching an entity to refresh it, it's worth noting that all related entities via navigations would need to be refreshed too manually – Xriuk Feb 23 '23 at 10:05
19

Reload and ReloadAsync has been available since Entity Framework Core 1.1

Examples:

//test.Name is test1
var test = dbContext.Tests.FirstOrDefault();
test.Name = "test2";

//test.Name is now test1 again
dbContext.Entry(test).Reload();

https://learn.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.changetracking.entityentry.reload?view=efcore-1.1

Ogglas
  • 62,132
  • 37
  • 328
  • 418
  • Two other answers already mention these methods. – Gert Arnold Jan 14 '21 at 15:52
  • 4
    @GertArnold Very true, I decided to post an answer given that OP stated `I only found people use Reload method, which is not available in EF core`. and that I thought a short and complete example was missing. – Ogglas Jan 14 '21 at 15:57
6

With the release of .NET 5.0 and Entity Framework Core 5.0, the recommended pattern would be to use a DBContext factory. In Statup.cs I changed:

services.AddDbContext<MyDbContext>...

to

services.AddDbContextFactory<MyDbContext>...

All my repository classes use the same base class, here I create the context in the constructor:

protected BaseRepository(IDbContextFactory<MyDbContext> contextFactory)
{
    _context = contextFactory.CreateDbContext();
}

I use a very simple repository factory to ensure I get a fresh instance of the repository and dbcontext each time I need it:

using System;
using Data.Models.Entities;
using Microsoft.Extensions.DependencyInjection;

namespace Data.Repository
{
    public class RepositoryFactory : IRepositoryFactory
    {
        private readonly IServiceProvider _serviceProvider;

        public RepositoryFactory(IServiceProvider serviceProvider)
        {
            _serviceProvider = serviceProvider;
        }
        
        public IApplicationRepository BuildApplicationRepository()
        {
            var service = _serviceProvider.GetService<IApplicationRepository>();

            return service;
        }
    }
}

Using the patterns described solves the "DB context is created by dependency injection" error and negates the need for a Reload()/Refresh() method.

James Blake
  • 1,050
  • 1
  • 14
  • 17
4

You would have to detach the entity from the context, or implement you own extension for .Reload()

Here's the .Reload() implementation. Source: https://weblogs.asp.net/ricardoperes/implementing-missing-features-in-entity-framework-core

public static TEntity Reload<TEntity>(this DbContext context, TEntity entity) where TEntity : class
{
    return context.Entry(entity).Reload();
}

public static TEntity Reload<TEntity>(this EntityEntry<TEntity> entry) where TEntity : class
{
    if (entry.State == EntityState.Detached)
    {
        return entry.Entity;
    }

    var context = entry.Context;
    var entity = entry.Entity;
    var keyValues = context.GetEntityKey(entity);

    entry.State = EntityState.Detached;

    var newEntity = context.Set<TEntity>().Find(keyValues);
    var newEntry = context.Entry(newEntity);

    foreach (var prop in newEntry.Metadata.GetProperties())
    {
        prop.GetSetter().SetClrValue(entity, 
        prop.GetGetter().GetClrValue(newEntity));
    }

    newEntry.State = EntityState.Detached;
    entry.State = EntityState.Unchanged;

    return entry.Entity;
}

Where GetEntityKey():

public static object[] GetEntityKey<T>(this DbContext context, T entity) where T : class
{
    var state = context.Entry(entity);
    var metadata = state.Metadata;
    var key = metadata.FindPrimaryKey();
    var props = key.Properties.ToArray();

    return props.Select(x => x.GetGetter().GetClrValue(entity)).ToArray();
}
penleychan
  • 5,370
  • 1
  • 17
  • 28
4

This simple sequence refreshes the whole context under EFCore 5.0:

public void Refresh()
{
    (Context as DbContext).Database.CloseConnection();
    (Context as DbContext).Database.OpenConnection();
}
wai-da
  • 49
  • 3
3

Here is my solution, I hope it will help you.

  • The detach function will detach all nested entities and array of entities.
  • The findByIdAsync will have an optional parameter to detach the entity and will reload it.

Repository

    public void Detach(TEntity entity)
    {
        foreach (var entry in _ctx.Entry(entity).Navigations)
        {
            if (entry.CurrentValue is IEnumerable<IEntity> children)
            {
                foreach (var child in children)
                {
                    _ctx.Entry(child).State = EntityState.Detached;
                }
            }
            else if (entry.CurrentValue is IEntity child)
            {
                _ctx.Entry(child).State = EntityState.Detached;
            }
        }
        _ctx.Entry(entity).State = EntityState.Detached;
    }

So for exemple with:

Classes:

public interface IEntity : IEntity<Guid>
{
}

public interface IEntity<TPrimaryKey>
{
    [JsonProperty("id")]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    TPrimaryKey Id { get; set; }
}

public class Sample : IEntity
{
    public Guid Id { get; set; }
    public string Text { get; private set; }   
    public Guid? CreatedByUserId { get; set; }
    public virtual User CreatedByUser { get; set; }     
    public List<SampleItem> SampleItems { get; set; } = new List<SampleItem>();
}

public class SampleItem : IEntity
{
    public Guid Id { get; set; }
    public string Text { get; private set; }   
    public Guid? CreatedByUserId { get; set; }
    public virtual User CreatedByUser { get; set; }     
}

Manager

    public async Task<Sample> FindByIdAsync(Guid id, bool includeDeleted = false, bool forceRefresh = false)
    {
        var result = await GetAll()
            .Include(s => s.SampleItems)
            .IgnoreQueryFilters(includeDeleted)
            .FirstOrDefaultAsync(s => s.Id == id);

        if (forceRefresh)
        {
            _sampleRepository.Detach(result);
            return await FindByIdAsync(id, includeDeleted);
        }

        return result;
    }

Controller

    SampleManager.FindByIdAsync(id, forceRefresh: true);
Cedric Arnould
  • 1,991
  • 4
  • 29
  • 49
-4

Ideally the other party would notify you when an update is made. This could be achieved via a message queue.

Among others: Azure Service Bus, Rabbit MQ, 0MQ

  • 4
    The OP is asking about how to force an EntityFramework `DbContext` to reload data into a loaded entity. If you do `var thing1 = myContext.Things.Find(1); /* something updates the database in the meantime */ var thing2 = myContext.Things.Find(1);`, then `thing2 == thing1` (same object instance) and the changes in the database will not be loaded into it because `DbContext` is using its cache instead of reloading the data. – binki Jul 11 '18 at 16:15