Aias00 opened a new pull request, #4976:
URL: https://github.com/apache/rocketmq-dashboard/pull/4976

   ## Summary
   
   - order AI timeline rows by `seq ASC` in SQL before applying the bounded 
limit
   - rely on the existing unique index `(conversation_id, seq)` for the range 
and ordering
   - remove the in-memory sort that could only order an already arbitrary 
limited subset
   - update repository, controller, service, test, and architecture 
documentation to preserve the pagination contract
   - keep page probing and `nextAfter` behavior unchanged
   
   Closes #4974
   
   ## Verification
   
   - `mvn -Dtest=AiTimelineRepositoryTest test` (35 passed)
   - `mvn -Dtest='org/apache/rocketmq/studio/ops/ai/**/*Test' test` (921 
passed, 3 skipped)
   - `mvn -DskipTests package` (passed, Checkstyle clean)
   - `git diff --check origin/rocketmq-studio...HEAD` (passed)
   - independent code review: approved after resolving the 
architecture-documentation finding
   
   ## Overlap check
   
   Open PR #4948 changes reconnect replay draining in `AiRunService`; it does 
not touch this repository query, SQL contract, or documentation surface.
   


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