Thanks Yufei for reviewing the pr, will take a look at your comments there.
Regards, Rahil Chertara From: Yufei Gu <flyrain...@gmail.com> Reply-To: "dev@iceberg.apache.org" <dev@iceberg.apache.org> Date: Friday, September 6, 2024 at 8:12 PM To: "hongyue_zh...@apple.com.invalid" <hongyue_zh...@apple.com.invalid> Cc: "dev@iceberg.apache.org" <dev@iceberg.apache.org> Subject: RE: [EXTERNAL] [VOTE] Merge REST Spec Change To Add New Scan Planning APIs CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe. Thanks Rahil for working on this. LGTM overall. Left minor comments. Yufei On Fri, Sep 6, 2024 at 1:22 PM Steve Zhang <hongyue_zh...@apple.com.invalid> wrote: Thank you Rahil for driving this, I left a comment as I am curious for #4 why do we consider it as POST call instead of get? Otherwise +1 1. An endpoint fetchScanTasks was added in order for a client to get the file-scan-tasks associated with a plan-task by providing a plan-task as input. Thanks, Steve Zhang On Sep 4, 2024, at 9:53 AM, Chertara, Rahil <rcher...@amazon.com.INVALID> wrote: 1. An endpoint fetchScanTasks was added in order for a client to get the file-scan-tasks associated with a plan-task by providing a plan-task as input.
