bug: ReorderSort does not work as expected

[creatorrr]

ReorderSort does not return anything. Am I doing something wrong? @zh217

Query:

test[a] <- [[1], [2], [3]]

?[i, x] <~ ReorderSort(test[a], out: [a], sort_by: [-a])

Expected:

i	x
1	3
2	2
3	1

Actual:

i	x

[wti]

Working for me with take: n != 0 e.g.,

test[a] <- [[1], [2], [3]]

?[i, x] <~ ReorderSort(test[a], out: [a], sort_by: [-a], take: 10)

So it looks like the default value for take of 0 does not have the documented magic significance of unbounded.

[wti]

See

cozo/cozo-core/src/fixed_rule/utilities/reorder_sort.rs

Line 112 in 6ab063c

if count > take_plus_skip {

perhaps:

if 0 != count && count > take_plus_skip {
    break;
}

[wti]

Proposed fix in PR #277

[creatorrr]

Ah that makes sense! Thanks for the catch and the fix 😄 Closing