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

Query: Add support for custom EntityQueryables #17001

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -167,6 +167,7 @@ public override EntityFrameworkServicesBuilder TryAddCoreServices()
TryAdd<IMethodCallTranslatorProvider, RelationalMethodCallTranslatorProvider>();
TryAdd<IMemberTranslatorProvider, RelationalMemberTranslatorProvider>();
TryAdd<IShapedQueryOptimizerFactory, RelationalShapedQueryOptimizerFactory>();
TryAdd<IQueryOptimizerFactory, RelationalQueryOptimizerFactory>();
TryAdd<IRelationalSqlTranslatingExpressionVisitorFactory, RelationalSqlTranslatingExpressionVisitorFactory>();
TryAdd<ISqlExpressionFactory, SqlExpressionFactory>();

Expand All @@ -188,6 +189,7 @@ public override EntityFrameworkServicesBuilder TryAddCoreServices()
.AddDependencySingleton<QuerySqlGeneratorDependencies>()
.AddDependencySingleton<RelationalShapedQueryCompilingExpressionVisitorDependencies>()
.AddDependencySingleton<RelationalShapedQueryOptimizerDependencies>()
.AddDependencySingleton<RelationalQueryOptimizerDependencies>()
.AddDependencyScoped<MigrationsSqlGeneratorDependencies>()
.AddDependencyScoped<RelationalConventionSetBuilderDependencies>()
.AddDependencyScoped<ModificationCommandBatchFactoryDependencies>()
Expand Down
46 changes: 46 additions & 0 deletions src/EFCore.Relational/Query/Internal/FromSqlEntityQueryable.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
// Copyright (c) .NET Foundation. All rights reserved.
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.

using System.Linq.Expressions;

namespace Microsoft.EntityFrameworkCore.Query.Internal
{
/// <summary>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
/// the same compatibility standards as public APIs. It may be changed or removed without notice in
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </summary>
public class FromSqlEntityQueryable<TResult> : EntityQueryable<TResult>
{
/// <summary>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
/// the same compatibility standards as public APIs. It may be changed or removed without notice in
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </summary>
public FromSqlEntityQueryable(IAsyncQueryProvider queryProvider, string sql, Expression arguments)
: base(queryProvider)
{
Sql = sql;
Arguments = arguments;
Expression = Expression.Constant(this);
}

/// <summary>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
/// the same compatibility standards as public APIs. It may be changed or removed without notice in
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </summary>
public virtual string Sql { get; }

/// <summary>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
/// the same compatibility standards as public APIs. It may be changed or removed without notice in
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </summary>
public virtual Expression Arguments { get; }
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
// Copyright (c) .NET Foundation. All rights reserved.
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.

using System.Linq;
using System.Linq.Expressions;
using System.Reflection;
using JetBrains.Annotations;

namespace Microsoft.EntityFrameworkCore.Query.Internal
{
public class FromSqlEntityQueryableInjectingExpressionVisitor : ExpressionVisitor
{
protected override Expression VisitMethodCall(MethodCallExpression methodCallExpression)
{
if (methodCallExpression.Method.DeclaringType == typeof(RelationalQueryableExtensions)
&& methodCallExpression.Method.Name == nameof(RelationalQueryableExtensions.FromSqlOnQueryable))
{
var sql = (string)((ConstantExpression)methodCallExpression.Arguments[1]).Value;
var queryable = (IQueryable)((ConstantExpression)methodCallExpression.Arguments[0]).Value;

return Expression.Constant(
_createFromSqlEntityQueryableMethod
.MakeGenericMethod(queryable.ElementType)
.Invoke(null, new object[] { queryable.Provider, sql, methodCallExpression.Arguments[2] }));
}

return base.VisitMethodCall(methodCallExpression);
}

private static readonly MethodInfo _createFromSqlEntityQueryableMethod
= typeof(FromSqlEntityQueryableInjectingExpressionVisitor)
.GetTypeInfo().GetDeclaredMethod(nameof(_CreateFromSqlEntityQueryable));

[UsedImplicitly]
// ReSharper disable once InconsistentNaming
private static FromSqlEntityQueryable<TResult> _CreateFromSqlEntityQueryable<TResult>(
IAsyncQueryProvider entityQueryProvider,
string sql,
Expression arguments)
=> new FromSqlEntityQueryable<TResult>(entityQueryProvider, sql, arguments);
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
// Copyright (c) .NET Foundation. All rights reserved.
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.

namespace Microsoft.EntityFrameworkCore.Query.Internal
{
public class RelationalQueryOptimizerFactory : IQueryOptimizerFactory
{
private readonly QueryOptimizerDependencies _dependencies;
private readonly RelationalQueryOptimizerDependencies _relationalDependencies;

public RelationalQueryOptimizerFactory(
QueryOptimizerDependencies dependencies,
RelationalQueryOptimizerDependencies relationalDependencies)
{
_dependencies = dependencies;
_relationalDependencies = relationalDependencies;
}

public virtual QueryOptimizer Create(QueryCompilationContext queryCompilationContext)
{
return new RelationalQueryOptimizer(
_dependencies,
_relationalDependencies,
queryCompilationContext);
}
}
}
29 changes: 29 additions & 0 deletions src/EFCore.Relational/Query/RelationalQueryOptimizer.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
// Copyright (c) .NET Foundation. All rights reserved.
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.

using System.Linq.Expressions;
using Microsoft.EntityFrameworkCore.Query.Internal;

namespace Microsoft.EntityFrameworkCore.Query
{
public class RelationalQueryOptimizer : QueryOptimizer
{
public RelationalQueryOptimizer(
QueryOptimizerDependencies dependencies,
RelationalQueryOptimizerDependencies relationalDependencies,
QueryCompilationContext queryCompilationContext)
: base(dependencies, queryCompilationContext)
{
RelationalDependencies = relationalDependencies;
}

protected virtual RelationalQueryOptimizerDependencies RelationalDependencies { get; }

public override Expression Visit(Expression query)
{
query = new FromSqlEntityQueryableInjectingExpressionVisitor().Visit(query);

return base.Visit(query);
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
// Copyright (c) .NET Foundation. All rights reserved.
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.

using Microsoft.EntityFrameworkCore.Infrastructure;
using Microsoft.Extensions.DependencyInjection;

namespace Microsoft.EntityFrameworkCore.Query
{
/// <summary>
/// <para>
/// Service dependencies parameter class for <see cref="RelationalShapedQueryOptimizer" />
/// </para>
/// <para>
/// This type is typically used by database providers (and other extensions). It is generally
/// not used in application code.
/// </para>
/// <para>
/// Do not construct instances of this class directly from either provider or application code as the
/// constructor signature may change as new dependencies are added. Instead, use this type in
/// your constructor so that an instance will be created and injected automatically by the
/// dependency injection container. To create an instance with some dependent services replaced,
/// first resolve the object from the dependency injection container, then replace selected
/// services using the 'With...' methods. Do not call the constructor at any point in this process.
/// </para>
/// <para>
/// The service lifetime is <see cref="ServiceLifetime.Scoped"/>. This means that each
/// <see cref="DbContext"/> instance will use its own instance of this service.
/// The implementation may depend on other services registered with any lifetime.
/// The implementation does not need to be thread-safe.
/// </para>
/// </summary>
public sealed class RelationalQueryOptimizerDependencies
{
/// <summary>
/// <para>
/// Creates the service dependencies parameter object for a <see cref="RelationalQueryOptimizer" />.
/// </para>
/// <para>
/// Do not call this constructor directly from either provider or application code as it may change
/// as new dependencies are added. Instead, use this type in your constructor so that an instance
/// will be created and injected automatically by the dependency injection container. To create
/// an instance with some dependent services replaced, first resolve the object from the dependency
/// injection container, then replace selected services using the 'With...' methods. Do not call
/// the constructor at any point in this process.
/// </para>
/// <para>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
/// the same compatibility standards as public APIs. It may be changed or removed without notice in
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </para>
/// </summary>
[EntityFrameworkInternal]
public RelationalQueryOptimizerDependencies()
{
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -61,17 +61,23 @@ private RelationalQueryableMethodTranslatingExpressionVisitor(
_sqlExpressionFactory = sqlExpressionFactory;
}

protected override Expression VisitMethodCall(MethodCallExpression methodCallExpression)
protected override Expression VisitConstant(ConstantExpression constantExpression)
=> constantExpression.Type.IsGenericType
&& constantExpression.Type.GetGenericTypeDefinition() == typeof(FromSqlEntityQueryable<>)
? (Expression)_createFromSqlShapedQueryExpressionMethodInfo
.MakeGenericMethod(constantExpression.Type.GetGenericArguments()[0])
.Invoke(this, new object[] { constantExpression })
: base.VisitConstant(constantExpression);

private static readonly MethodInfo _createFromSqlShapedQueryExpressionMethodInfo
= typeof(RelationalQueryableMethodTranslatingExpressionVisitor)
.GetTypeInfo().GetDeclaredMethod(nameof(CreateFromSqlShapedQueryExpression));

private Expression CreateFromSqlShapedQueryExpression<TResult>(ConstantExpression constantExpression)
{
if (methodCallExpression.Method.DeclaringType == typeof(RelationalQueryableExtensions)
&& methodCallExpression.Method.Name == nameof(RelationalQueryableExtensions.FromSqlOnQueryable))
{
var sql = (string)((ConstantExpression)methodCallExpression.Arguments[1]).Value;
var queryable = (IQueryable)((ConstantExpression)methodCallExpression.Arguments[0]).Value;
return CreateShapedQueryExpression(queryable.ElementType, sql, methodCallExpression.Arguments[2]);
}

return base.VisitMethodCall(methodCallExpression);
var fromSqlEntityQueryable = (FromSqlEntityQueryable<TResult>)constantExpression.Value;
return CreateShapedQueryExpression(
fromSqlEntityQueryable.ElementType, fromSqlEntityQueryable.Sql, fromSqlEntityQueryable.Arguments);
}

public override ShapedQueryExpression TranslateSubquery(Expression expression)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,9 @@ public sealed class RelationalShapedQueryOptimizerDependencies
public RelationalShapedQueryOptimizerDependencies(
[NotNull] ISqlExpressionFactory sqlExpressionFactory)
{
SqlExpressionFactory = sqlExpressionFactory;
Check.NotNull(sqlExpressionFactory, nameof(sqlExpressionFactory));

SqlExpressionFactory = sqlExpressionFactory;
}

/// <summary>
Expand Down
20 changes: 18 additions & 2 deletions src/EFCore/Extensions/Internal/ExpressionExtensions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -142,8 +142,24 @@ public static bool IsLogicalOperation([NotNull] this Expression expression)
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </summary>
public static bool IsEntityQueryable([NotNull] this ConstantExpression constantExpression)
=> constantExpression.Type.GetTypeInfo().IsGenericType
&& constantExpression.Type.GetGenericTypeDefinition() == typeof(EntityQueryable<>);
{
if (constantExpression.Type.GetTypeInfo().IsGenericType)
{
if (constantExpression.Type.GetTypeInfo().GetGenericTypeDefinition() == typeof(EntityQueryable<>))
{
return true;
}

var genericArguments = constantExpression.Type.GetTypeInfo().GetGenericArguments();
if (genericArguments.Length == 1)
{
return typeof(EntityQueryable<>).MakeGenericType(genericArguments[0])
.GetTypeInfo().IsAssignableFrom(constantExpression.Type);
}
}

return false;
}

/// <summary>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
Expand Down
2 changes: 1 addition & 1 deletion src/EFCore/Query/Internal/EntityQueryable`.cs
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ public EntityQueryable([NotNull] IAsyncQueryProvider queryProvider, [NotNull] Ex
/// any release. You should only use it directly in your code with extreme caution and knowing that
/// doing so can result in application failures when updating to a new Entity Framework Core release.
/// </summary>
public virtual Expression Expression { get; }
public virtual Expression Expression { get; protected set; }

/// <summary>
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to
Expand Down