From 92bc2ad89fa6f7abb1d08a4802ca4767794ac76c Mon Sep 17 00:00:00 2001 From: Kushal Agrawal Date: Thu, 26 Apr 2018 12:15:46 +0530 Subject: [PATCH] list api with limit and search text issue If artifact data has multiple properties other than base artifact properties i.e. which needs to be stored in glare_artifact_properties table. In that case providing limit will always result in less no of artifacts then limit. This issue is not available in sqllite database, therefore no additional testcase is added. Change-Id: I933783cabb5b71d92b4d645d340bd9d1c9f6f8b5 closes-bug: #1766537 --- glare/db/sqlalchemy/api.py | 1 + 1 file changed, 1 insertion(+) diff --git a/glare/db/sqlalchemy/api.py b/glare/db/sqlalchemy/api.py index 6ed2b54..c5aab88 100644 --- a/glare/db/sqlalchemy/api.py +++ b/glare/db/sqlalchemy/api.py @@ -390,6 +390,7 @@ def _do_paginate_query(query, marker=None, limit=None, sort=None): query = query.filter(or_(*criteria_list)) if limit is not None: + query = query.group_by(models.Artifact.id) query = query.limit(limit) return query