mengw15 opened a new pull request, #8713: URL: https://github.com/apache/texera/pull/8713
### What changes were proposed in this PR? Flips `storage.warehouse.enabled` to `true`, enabling the per-user warehouse feature that has been built behind it. All of #6870's Phase 0 work is merged; the switch is the last step. The comment above it is updated too — it described a feature still being built. The switch only takes effect where the Iceberg catalog is `rest` (Lakekeeper) — `StorageConfig` keeps the feature off for any other catalog type, so deployments that do not run Lakekeeper are unaffected. ### Any related issues, documentation, discussions? Closes #8712. Part of #6870. Proposed and demonstrated in discussion #8633, with no objections raised. ### How was this PR tested? - Ran the engine's integration suite locally against a live Lakekeeper and MinIO, with the switch on and off. The same eight cases fail either way (Python UDF specs timing out in my environment), so the switch makes no difference to the suite. The `amber` unit job pins `STORAGE_ICEBERG_CATALOG_TYPE: postgres`, which keeps the feature off there regardless of this flag. - Exercised the feature by hand on a local deployment with the switch on: creating a warehouse from the dashboard tab and from the canvas picker, running a workflow into it, and confirming that a run with no warehouse selected is refused rather than routed to shared storage. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (claude-opus-5, claude-fable-5) -- 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]
