laserninja commented on PR #12194: URL: https://github.com/apache/gravitino/pull/12194#issuecomment-5246735447
Correction to the plan above: **slice 2 is not needed, credential vending for `POST .../plan` is already in `main`.** I had been comparing against a stale fork main, 132 commits behind, which is also what broke CI on #12411 - the merge with the real `main` placed a hunk outside the scope it needed, because `planTableScan` has since been restructured. Rebased and fixed there. So the stack is four, not five: | # | PR | What it contains | | - | -- | ---------------- | | 1 | #12411 | Order planned file scan tasks deterministically | | 2 | to open | The `POST .../tasks` endpoint and its plumbing: dispatcher chain, events, audit operation type, `NoSuchPlanTaskException` → `404`. Not advertised in `/v1/config` yet | | 3 | to open | Federated catalogs forward `POST .../tasks` to the remote catalog | | 4 | this PR | Batching a plan into `plan-tasks`, `PlanTaskCodec`, `scan-plan-task-batch-size`, advertising the endpoint in `/v1/config`, user docs | The ordering constraints are unchanged: the `/v1/config` advertisement lands last so the endpoint is never advertised before it can serve tasks, and federation lands before batching so a federated deployment never advertises an endpoint it would 404 on. I am rebasing this PR onto current `main` now, so that what you see here is a diff against the base it will actually merge into rather than one 132 commits behind. -- 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]
