Re: [PR] Support more reliable async task retry to guarantee eventual execution (2/2) – Task Executor [polaris]

["danielhumanmod (via GitHub)" <gi...@apache.org>]

danielhumanmod commented on code in PR #1585:
URL: https://github.com/apache/polaris/pull/1585#discussion_r2094263997


##########
quarkus/service/src/test/java/org/apache/polaris/service/quarkus/task/TableCleanupTaskHandlerTest.java:
##########
@@ -152,6 +156,7 @@ public void testTableCleanup() throws IOException {
 
     handler.handleTask(task, callContext);
 
+    timeSource.add(Duration.ofMinutes(10));

Review Comment:
   Previously, task entity might miss `LAST_ATTEMPT_START_TIME` prop so loading tasks without time-out can success; After complete each task entity with this property, we need to manipulate time to make `loadTasks` works



-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org