mengw15 opened a new pull request, #7359: URL: https://github.com/apache/texera/pull/7359
### What changes were proposed in this PR? First switch of the per-user warehouse feature (umbrella #6870): a new `storage.warehouse.enabled` config key (default **off**, overridable via `STORAGE_WAREHOUSE_ENABLED`) and the derived gate `StorageConfig.warehousesEnabled`, which is on only when the switch is on **and** the deployment uses the REST (Lakekeeper) catalog — per-user warehouses are Lakekeeper entities, so any other catalog type keeps the feature off regardless of the switch. On its own this changes nothing: nothing reads the gate yet. The follow-up PRs check `warehousesEnabled` to decide whether to expose the warehouse API/UI and route executions; the explicit read-path failure semantics described in #6930 land with those PRs, since they need the warehouse-aware read path from #6944. ### Any related issues, documentation, discussions? Closes #6930. Part of #6870 (design discussions #5293 and #6040). ### How was this PR tested? `StorageConfigSpec` gains three cases: the default stays off (guarded on the env override being unset), the derived gate's truth table, and the env-var override name. Verified locally with `sbt "Config/testOnly *StorageConfigSpec"` plus scalafmt/scalafix checks; the truth-table case was deliberately broken once to confirm it fails red. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (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]
