sergehuber opened a new pull request, #917:
URL: https://github.com/apache/unomi/pull/917

   ## Summary
   - Sort the merged system+tenant segment and scoring metadata lists before 
paging, using an allowlisted set of fields (id, name, scope, dates, …); unknown 
sort fields are refused.
   - Treat a missing or negative query limit as “all”, capped by the same 
maximum page size used elsewhere, so `POST /cxs/segments/query` and `POST 
/cxs/scoring/query` no longer fail when limit is unset or `-1`.
   - Unit tests cover sort (asc/desc by name and date), paging variants, 
unknown fields, and the max page size cap for both segments and scorings across 
system and tenant items; integration tests cover POST query with no limit for 
both endpoints.
   
   ## Test plan
   - [x] Unit tests for MergedMetadataListHelper and SegmentServiceImpl merged 
list sort/paging (segments and scorings)
   - [ ] CI integration tests 
(`SegmentIT.testQuerySegmentsWithNoLimitReturnsFullList`, 
`SegmentIT.testQueryScoringsWithNoLimitReturnsFullList`)
   - [ ] Manual: create several segments/scorings, query with sort, with no 
limit, and with limit `-1`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to