huaxingao commented on code in PR #14730:
URL: https://github.com/apache/iceberg/pull/14730#discussion_r2578950387
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -824,6 +828,8 @@ paths:
- Catalog API
summary: Fetches result tasks for a plan task
operationId: fetchScanTasks
+ parameters:
+ - $ref: '#/components/parameters/idempotency-key'
Review Comment:
Thanks for thinking through the async behavior. Conceptually that model
makes sense; however, in the current implementation both sync and async
planning call planFilesFor, which fully materializes plan tasks and wires the
next links during planning. fetchScanTasks is therefore a read‑only lookup by
planTask and will return the same batch/next on retry. Given that, probably we
don't need to add an Idempotency‑Key to fetchScanTasks right now. If we later
move to a lazy/adaptive planning model, we can revisit this. wdyt?
--
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]
