Commit 93835a3
committed
docs(#373): verify sortBy already implemented for VectorQuery and VectorRangeQuery
Issue #373 requested adding sortBy support to VectorQuery and
VectorRangeQuery. Investigation reveals this feature is already fully
implemented:
Implementation:
- VectorQuery: sortBy() and sortDescending() methods (lines 817-831)
- VectorRangeQuery: sortBy() and sortDescending() methods (lines 367-381)
- SearchIndex.searchWithSort() applies sorting via Jedis (lines 1360-1399)
- Sorting parameters properly passed through query execution pipeline
Test Coverage:
- QuerySortingIntegrationTest with 4 passing tests
- testSortVectorQuery: ascending sort verification
- testSortVectorQueryDescending: descending sort verification
- testSortVectorRangeQuery: range query sorting
- testSortFilterQueryAlreadyWorks: baseline comparison
Changes:
- Removed outdated test comments claiming methods don't exist
- No implementation changes needed
Tests: 282 passed, 14 skipped (API keys)
Closes #373 - feature already complete with full test coverage1 parent 87bca26 commit 93835a3
File tree
1 file changed
+2
-2
lines changed- core/src/test/java/com/redis/vl/query
1 file changed
+2
-2
lines changedLines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
179 | 179 | | |
180 | 180 | | |
181 | 181 | | |
182 | | - | |
| 182 | + | |
183 | 183 | | |
184 | 184 | | |
185 | 185 | | |
| |||
244 | 244 | | |
245 | 245 | | |
246 | 246 | | |
247 | | - | |
| 247 | + | |
248 | 248 | | |
249 | 249 | | |
250 | 250 | | |
| |||
0 commit comments