Merge pull request #1097 from jakedt/snowflake

Improve performance by removing unnecessary group by fields

data/model/repository.py

@@ -294,10 +294,10 @@ def get_sorted_matching_repositories(prefix, only_public, checker, limit=10):
 
     query = (Repository
              .select(*select_items)
-             .join(Namespace, JOIN_LEFT_OUTER, on=(Namespace.id == Repository.namespace_user))
+             .join(Namespace, on=(Namespace.id == Repository.namespace_user))
              .switch(Repository)
              .where(search_clause)
-             .group_by(Repository, Namespace))
+             .group_by(Repository.id, Namespace.id))
 
     if only_public:
       query = query.where(Repository.visibility == _basequery.get_public_repo_visibility())

data/model/storage.py

@@ -48,8 +48,8 @@ def garbage_collect_storage(storage_id_whitelist):
                                                        .switch(ImageStoragePlacement)
                                                        .join(ImageStorage),
                                                        storage_id_whitelist,
-                                                       (ImageStorage, ImageStoragePlacement,
-                                                        ImageStorageLocation)))
+                                                       (ImageStorage.id, ImageStoragePlacement.id,
+                                                        ImageStorageLocation.id)))
 
     paths_to_remove = placements_query_to_paths_set(placements_to_remove)