Skip to content

Fix: unstable sort skips/duplicates issues across pages#18095

Merged
lunny merged 3 commits intogo-gitea:release/v1.15from
DuckDuckWhale:release/v1.15
Dec 29, 2021
Merged

Fix: unstable sort skips/duplicates issues across pages#18095
lunny merged 3 commits intogo-gitea:release/v1.15from
DuckDuckWhale:release/v1.15

Conversation

@DuckDuckWhale
Copy link
Copy Markdown
Contributor

When viewing issues in sorted order, some issues are duplicated across pages and some are missing. This is caused by the lack of tie-breakers in database queries, making pagination inconsistent.

Backport of #18094.

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Dec 24, 2021
Comment thread models/issue.go Outdated
Comment thread models/issue.go Outdated
When viewing issues in sorted order, some issues are duplicated across
pages and some are missing.  This is caused by the lack of tie-breakers
in database queries, making pagination inconsistent.
@wxiaoguang wxiaoguang mentioned this pull request Dec 29, 2021
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 29, 2021
@lunny lunny merged commit 484fe07 into go-gitea:release/v1.15 Dec 29, 2021
@DuckDuckWhale DuckDuckWhale deleted the release/v1.15 branch December 29, 2021 22:11
@6543 6543 added this to the 1.15.9 milestone Dec 30, 2021
@6543 6543 added the type/bug label Dec 30, 2021
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants