Skip to content

Commit b376d6e

Browse files
committed
Merge pull request #519 from rogpeppe/168-more-indexes
internal/charmstore: add more indexes This should solve some performance issues.
2 parents ef8e5e8 + 9d697a8 commit b376d6e

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

internal/charmstore/store.go

+10
Original file line numberDiff line numberDiff line change
@@ -390,9 +390,19 @@ func (s *Store) ensureIndexes() error {
390390
}, {
391391
s.DB.Entities(),
392392
mgo.Index{Key: []string{"bundlecharms"}},
393+
}, {
394+
s.DB.Entities(),
395+
mgo.Index{Key: []string{"name", "development", "-promulgated-revision", "-supportedseries"}},
396+
}, {
397+
s.DB.Entities(),
398+
mgo.Index{Key: []string{"name", "development", "user", "-revision", "-supportedseries"}},
393399
}, {
394400
s.DB.BaseEntities(),
395401
mgo.Index{Key: []string{"name"}},
402+
}, {
403+
// TODO this index should be created by the mgo gridfs code.
404+
s.DB.C("entitystore.files"),
405+
mgo.Index{Key: []string{"filename"}},
396406
}}
397407
for _, idx := range indexes {
398408
err := idx.c.EnsureIndex(idx.i)

internal/charmstore/store_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -1542,7 +1542,7 @@ func (s *StoreSuite) TestCollections(c *gc.C) {
15421542
}
15431543
// Check that all created collections are mentioned in Collections.
15441544
for _, name := range names {
1545-
if name == "system.indexes" || name == "managedStoredResources" {
1545+
if name == "system.indexes" || name == "managedStoredResources" || name == "entitystore.files" {
15461546
continue
15471547
}
15481548
found := false

0 commit comments

Comments
 (0)