shangeyao opened a new pull request, #4391:
URL: https://github.com/apache/streampark/pull/4391

   ## Summary
   - Load all application backups in one query ordered by create time
   - Group by app id in memory and collect ids beyond the retention limit
   - Delete expired backups with a single `removeByIds` call
   
   Fixes #4390
   
   ## Test plan
   - [ ] Seed more than `max-backup-num` backups for an app and run cleanup
   - [ ] Verify only oldest extras are removed and latest N are kept
   
   ---
   **AI Disclosure**
   - Model: Claude Opus 4.6
   - Platform/Tool: Cursor
   - Human Oversight: partially reviewed
   - Prompt Summary: Optimize backup cleanup task from dev branch scan
   
   
   Made with [Cursor](https://cursor.com)


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