[x] The commit(s) message(s) follows the contribution guidelines ?
[x] Tests for the changes have been added (for bug fixes / features) ?
[ ] Docs have been added / updated (for bug fixes / features) ?
Current behavior :
When converting streaming Query response to a single list we assemble the range vectors in following manner
Consider we get the following range vector in batches serialized
[[1, 2, 3], [4, 5, 6], [7, 8]]
When assembling the result becomes
[[7, 8], [4, 5, 6], [1, 2, 3]]
This was due to the fact we used the cons operator (::) to prepend the latest streamed response essentially reversing the order of these batched response. We thus see issues when operations like sort_desc where the response is strictly ordered but goes out of order in batches.
New behavior :
The cons operator (::) is removed and Seq to preserve the ordering. The unit test reproduces this phenomena and validates the fix.
Pull Request checklist
Current behavior :
When converting streaming Query response to a single list we assemble the range vectors in following manner
Consider we get the following range vector in batches serialized
[[1, 2, 3], [4, 5, 6], [7, 8]]
When assembling the result becomes
[[7, 8], [4, 5, 6], [1, 2, 3]]
This was due to the fact we used the cons operator (
::
) to prepend the latest streamed response essentially reversing the order of these batched response. We thus see issues when operations likesort_desc
where the response is strictly ordered but goes out of order in batches.New behavior :
The cons operator (
::
) is removed andSeq
to preserve the ordering. The unit test reproduces this phenomena and validates the fix.BREAKING CHANGES
None