geruh commented on code in PR #14660:
URL: https://github.com/apache/iceberg/pull/14660#discussion_r2554419678
##########
core/src/main/java/org/apache/iceberg/rest/CatalogHandlers.java:
##########
@@ -769,16 +769,22 @@ private static void planFilesFor(TableScan tableScan,
String planId, int tasksPe
Iterables.partition(tableScan.planFiles(), tasksPerPlanTask);
int planTaskSequence = 0;
String previousPlanTask = null;
+ String planTaskKeyPrefix = planId + "-" + tableScan.table().uuid() + "-";
Review Comment:
Yeha makes sense. Looks like you kept the logic in `planFilesFor` so sync
and async behaviors both still use the same planning state logic, without
changing the async fetch behavior yet. Let me take a look at this and push up a
revision
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
