Merge pull request #147 from ipfs/fix/optimize-naive-query

fix: avoid filtering by prefix unless necessary

Merge pull request #147 from ipfs/fix/optimize-naive-query
fix: avoid filtering by prefix unless necessary
d099e8e2 · Steven Allen · GitHub · ac8b6e68 · ec27f419 · d099e8e2
Unverified Commit d099e8e2 authored Feb 14, 2020 by Steven Allen Committed by GitHub Feb 14, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

query/query_impl.go query/query_impl.go +2 -4

No files found.
--- a/query/query_impl.go
+++ b/query/query_impl.go
@@ -129,12 +129,10 @@ func NaiveQueryApply(q Query, qr Results) Results {
 			}
 			prefix = path.Clean(prefix)
 		}
-		// If the prefix isn't "/", end it in a "/" so we only find keys
-		// _under_ the prefix.
+		// If the prefix is empty, ignore it.
 		if prefix != "/" {
-			prefix += "/"
+			qr = NaiveFilter(qr, FilterKeyPrefix{prefix + "/"})
 		}
-		qr = NaiveFilter(qr, FilterKeyPrefix{prefix})
 	}
 	for _, f := range q.Filters {
 		qr = NaiveFilter(qr, f)