You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/11/04 11:48:53 UTC

[3/3] git commit: updated refs/heads/master to 5f61041

Merge pull request #1009 from rags22489664/master

CLOUDSTACK-9006 - ListTemplates API returns result in inconsistent order when called concurrentlyThe order of templates returned in the response is based on a field called sortkey and by default value for the field is set to 0.

With more than 1000 templates, we tried listing the templates with different page sizes concurrently, and we noticed the results being inconsistent.

Thus we added a secondary order by clause to list templates call on tempZonePair column to make sure the results are consistent.

The addOrderby method of Filter class was also not appending , if we added more orderby clauses.

* pr/1009:
  CLOUDSTACK-9006 - ListTemplates API returns result in inconsistent order when called concurrently
  CLOUDSTACK-9006 - ListTemplates API returns result in inconsistent order when called concurrently

Signed-off-by: Remi Bergsma <gi...@remi.nl>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/5f610415
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/5f610415
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/5f610415

Branch: refs/heads/master
Commit: 5f610415504b9f18f91169761598a1585ce429c7
Parents: 2fd6fac 9d1a469
Author: Remi Bergsma <gi...@remi.nl>
Authored: Wed Nov 4 11:48:07 2015 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Wed Nov 4 11:48:08 2015 +0100

----------------------------------------------------------------------
 framework/db/src/com/cloud/utils/db/Filter.java |  2 +-
 .../db/test/com/cloud/utils/db/FilterTest.java  | 44 ++++++++++++++++++++
 .../com/cloud/api/query/QueryManagerImpl.java   |  1 +
 .../api/query/dao/TemplateJoinDaoImpl.java      |  1 +
 4 files changed, 47 insertions(+), 1 deletion(-)
----------------------------------------------------------------------