Revanth14 opened a new pull request, #2261: URL: https://github.com/apache/iceberg-go/pull/2261
### Problem `compaction.Analyze` plans with `tbl.Scan().PlanFiles` and never closes the scan. When a REST catalog plans the scan remotely and vends plan-scoped credentials, the planner keeps the server-side plan alive until the scan's plan IO is closed, so each analysis leaves an active plan on the server. Since #2162, every `compact` against a table whose catalog sets `scan-planning-mode=server` plans remotely. ### Fix `Analyze` closes its scan before returning. The returned tasks are always read through a separate scan (`table.ExecuteCompactionGroup` builds its own), so nothing needs the planning scan afterwards. ### Testing `TestAnalyze_ClosesRemotePlan` plans through a fake remote planner that returns a plan IO and checks that `Analyze` closes it. It fails without the fix. -- 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]
