Entity Framework + LINQ + "Enthält" == Super Slow?

Question

Ich versuche, Code neu zu gestalten, der in letzter Zeit sehr langsam geworden ist, und ich stieß auf einen Codeblock, der mehr als 5 Sekunden für die Ausführung benötigt.

Der Code besteht aus zwei Aussagen:

IEnumerable<int> StudentIds = _entities.Filters 
        .Where(x => x.TeacherId == Profile.TeacherId.Value && x.StudentId != null) 
        .Select(x => x.StudentId) 
        .Distinct<int>(); 

und

_entities.StudentClassrooms 
        .Include("ClassroomTerm.Classroom.School.District") 
        .Include("ClassroomTerm.Teacher.Profile") 
        .Include("Student") 
        .Where(x => StudentIds.Contains(x.StudentId) 
        && x.ClassroomTerm.IsActive 
        && x.ClassroomTerm.Classroom.IsActive 
        && x.ClassroomTerm.Classroom.School.IsActive 
        && x.ClassroomTerm.Classroom.School.District.IsActive).AsQueryable<StudentClassroom>(); 

So ist es ein bisschen chaotisch, aber zuerst bekomme ich eine Distinct Liste von IDs aus einer Tabelle (Filter), dann ich abfragen eine andere Tabelle, die es benutzt.

Dies sind relativ kleine Tabellen, aber es ist immer noch 5+ Sekunden Abfragezeit.

Ich legte dies in LINQPad und es zeigte, dass es zuerst die untere Abfrage und dann 1000 "distinct" Abfragen später ausgeführt wurde.

Aus einer Laune heraus änderte ich den "StudentIds" -Code, indem ich am Ende nur .ToArray() hinzufügte. Dies hat die Geschwindigkeit 1000x verbessert ... es dauert jetzt 100ms, um die gleiche Abfrage zu vervollständigen.

Was ist das Geschäft? Was mache ich falsch?

Answer 1

Dies ist eine der Fallstricke der verzögerten Ausführung in Linq: In Ihrem ersten Ansatz StudentIds ist wirklich eine IQueryable, keine In-Memory-Sammlung. Das heißt, wenn Sie es in der zweiten Abfrage verwenden, wird die Abfrage erneut in der Datenbank ausgeführt - jedes Mal.

Erzwingen der Ausführung der ersten Abfrage von ToArray() Verwendung macht StudentIds eine speicherinterne Sammlung und den Contains Teil in Ihre zweite Abfrage wird diese Sammlung überfahren, die eine festgelegte Abfolge von Elementen enthält - Dies wird äquivalent zu einer SQL etwas kartiert where StudentId in (1,2,3,4) Abfrage.

Diese Abfrage wird natürlich viel viel schneller sein, da Sie diese Sequenz einmal im Voraus bestimmt haben, und nicht jedes Mal, wenn die Where-Klausel ausgeführt wird. Ihre zweite Abfrage ohne Verwendung von ToArray() (würde ich denken) würde einer SQL-Abfrage mit einer where exists (...) Unterabfrage zugeordnet werden, die für jede Zeile ausgewertet wird.

Answer 2

ToArray() Verarbeitet die anfängliche Abfrage zum Serverspeicher.

Meine Vermutung wäre, dass der Abfrageanbieter den Ausdruck StudentIds.Contains(x.StudentId) nicht parsen kann. Daher denkt es wahrscheinlich, dass das studentIds ein Array ist, das bereits in den Speicher geladen wurde. Es wird also wahrscheinlich während der Analysephase die Datenbank immer wieder abgefragt. Der einzige Weg, um sicher zu wissen, ist, den Profiler einzurichten.

Wenn Sie dies auf dem Datenbankserver tun müssen, verwenden Sie einen Join statt "enthält". Wenn Sie contains verwenden müssen, um etwas zu tun, das wie ein Join-Problem aussieht, fehlt Ihnen wahrscheinlich irgendwo ein Ersatz-Primärschlüssel oder ein Fremdschlüssel.

Sie könnten auch studentIds als IQueryable anstelle von IEnumerable deklarieren. Dies kann dem Abfrageanbieter den Hinweis geben, der benötigt wird, um die studentIds als Ausdruck aka zu interpretieren. Daten nicht bereits in den Speicher geladen. Ich bezweifle das irgendwie, aber einen Versuch wert.

Wenn alles andere fehlschlägt, verwenden Sie ToArray(). Dies lädt die ursprüngliche studentIds in den Speicher.