Use AsNoTracking on Readonly queries

[jaredcnance]

https://docs.microsoft.com/en-us/ef/core/querying/tracking

[bart-degreed]

Recent changes in EF Core have resulted in AsNoTracking() being faster for simple queries, but slower for complex queries (described at dotnet/efcore#14366).

[bart-degreed]

As noted above, it depends on use cases whether or not it results in improvements, so I don't think we should have this built-in.

Nevertheless, it can easily be added by library users themselves:

public class NoTrackingRepository<TResource, TId> : EntityFrameworkCoreRepository<TResource, TId>
    where TResource : class, IIdentifiable<TId>
{
    private readonly IJsonApiRequest _request;

    public NoTrackingRepository(IJsonApiRequest request, ITargetedFields targetedFields,
        IDbContextResolver contextResolver, IResourceGraph resourceGraph, IResourceFactory resourceFactory,
        IEnumerable<IQueryConstraintProvider> constraintProviders, ILoggerFactory loggerFactory)
        : base(targetedFields, contextResolver, resourceGraph, resourceFactory, constraintProviders, loggerFactory)
    {
        _request = request;
    }

    protected override IQueryable<TResource> GetAll()
    {
        if (_request.IsReadOnly)
        {
            return base.GetAll().AsNoTracking();
        }

        return base.GetAll();
    }
}