Skip to content

Commit 6121e9e

Browse files
authored
Fix comment in testDoesNotAutoCreateIndexesWhenIndexLookUpIsExpensive
1 parent 6b2a90f commit 6121e9e

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

firebase-firestore/src/test/java/com/google/firebase/firestore/local/SQLiteLocalStoreTest.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -444,9 +444,7 @@ public void testDoesNotAutoCreateIndexesWhenIndexLookUpIsExpensive() {
444444
applyRemoteEvent(
445445
addedRemoteEvent(doc("coll/e", 10, map("array", Arrays.asList(2, 0, 8))), targetId));
446446

447-
// First time query runs without indexes.
448-
// Based on current heuristic, collection document counts (5) > 2 * resultSize (2).
449-
// Full matched index should be created.
447+
// SDK will not create indexes since relative read cost is too large.
450448
executeQuery(query);
451449
assertRemoteDocumentsRead(/* byKey= */ 0, /* byCollection= */ 2);
452450
assertQueryReturned("coll/a", "coll/e");

0 commit comments

Comments
 (0)