j1wonpark opened a new pull request, #4114: URL: https://github.com/apache/amoro/pull/4114
## Summary - Fix completeProcess(false) unconditionally updating lastOptimizedSnapshotId and lastOptimizedChangeSnapshotId, which caused SchedulingPolicy.isTablePending() to return false and permanently skip the table from re-planning - Only update snapshot markers when success is true (actual optimization was performed) - Add two tests: one verifying snapshot markers are preserved on process close, another verifying the table can be rescheduled after a failed process Closes #4113 ## How was this patch tested? - TestOptimizingQueue#testProcessCloseKeepsLastOptimizedSnapshotId - verifies that lastOptimizedSnapshotId and lastOptimizedChangeSnapshotId are not updated when process.close(false) is called - TestOptimizingQueue#testTableRescheduledAfterProcessClose - end-to-end test verifying the table transitions back to PENDING and is successfully re-planned after a forced process close -- 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]
