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

   ## Summary
   Fixes [UNOMI-1021](https://issues.apache.org/jira/browse/UNOMI-1021): a date 
aggregate with an unsupported interval (such as `2d`) returned an empty 204.
   
   - `DateAggregate` now rejects anything other than `1s` … `1y` and `Second` … 
`Year` with an `IllegalArgumentException`, which REST maps to 400. The log 
names the accepted values; the response body stays the generic `badRequest`, as 
for other bad requests.
   - This covers both places that build date aggregates from requests: 
`QueryServiceImpl` and `GoalsServiceImpl`.
   - Tests: `DateAggregateTest` (unit) and a new `AggregateQueryIT` (`2d` and 
`day` → 400, `1d` and `Day` → 200), registered in `AllITs` and 
`CorePersistenceITs`.
   
   ## Test plan
   - [x] `persistence-spi` unit tests (153)
   - [x] `AggregateQueryIT` on Elasticsearch and OpenSearch (2/2)
   - [ ] CI
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
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