veeso / ic-dbms

96%

main: 96%

LAST BUILD ON BRANCH fix/multi-column-order-by

branch: fix/multi-column-order-by

CHANGE BRANCH

x

Reset

Sync Branches

• fix/multi-column-order-by
• 0.3
• 0.4
• 14-add-value-from-all-inner-values
• 19-free-segments-ledger-can-be-bigger-than-a-page
• 3-field-validation
• 5-sanitizer
• 7-align-record-table-memory
• 9-missing-int-values
• build/25-pocket-ic-12
• build/36-get-rid-of-the-like-crate
• chore/deps-template
• chore/git-cliff
• ci/codeberg-mirror
• docs/reorganize-documentation
• feat/10-create-a-generic-select-method
• feat/12-join
• feat/13-json-type
• feat/23-make-ic-dbms-canister-client-testable
• main
• perf/11-update-instead-of-delete-and-insert
• refactor/dbms-cleanup
• refactor/move-crates-to-crates-folder
• test-memory-align-missing-checks
• test/increase-coverage-api-canister
• v0.1.0
• v0.2.0
• v0.2.1
• v0.3.0
• v0.4.0

Build # 21728364696

Build Type

Pull #39

github

Committed by veeso

Commit Message

fix(canister): apply multi-column order_by sorts in correct order

The order_by loop was applying each sort column sequentially, which
meant only the last column's sort survived. Since Rust's sort_by is
stable, reversing the iteration order (least-significant column first)
produces correct multi-column ordering.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Pull Request Pull Request #39: fix(canister): apply multi-column order_by sorts in correct order

Run Details

33 of 33 new or added lines in 1 file covered. (100.0%)

9798 of 10243 relevant lines covered (95.66%)

732978.61 hits per line