ClearML/clearml-server
1
0
Fork 0
mirror of https://github.com/clearml/clearml-server synced 2025-02-07 13:33:42 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Make sure id field is included in ordering

Browse Source
This commit is contained in:
allegroai 2022-11-29 17:40:09 +02:00
parent 58d9a64537
commit 9e0adc77dd
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
apiserver/database/model/base.py
View File

@ -522,6 +522,8 @@ class GetMixin(PropsMixin):
def validate_order_by(cls, parameters, search_text) -> Sequence: def validate_order_by(cls, parameters, search_text) -> Sequence:
""" """
Validate and extract order_by params as a list Validate and extract order_by params as a list
If ordering is specified then make sure that id field is part of it
This guarantees the unique order when paging
""" """
order_by = parameters.get(cls._ordering_key) order_by = parameters.get(cls._ordering_key)
if not order_by: if not order_by:
@ -534,6 +536,9 @@ class GetMixin(PropsMixin):
"text score cannot be used in order_by when search text is not used" "text score cannot be used in order_by when search text is not used"
) )
if not any(id_field in order_by for id_field in ("id", "-id")):
order_by.append("id")
return order_by return order_by
@classmethod @classmethod