dab: delete bundle/sql/ — no longer deployment artifacts #228

Closed
opened 2026-03-25 04:00:41 +00:00 by kert · 2 comments
Owner

Problem

208 SQL files in bundle/sql/ are no longer referenced by databricks.yml tasks (switched to PythonWheelTask in #223). The files remain in the repo as dead weight.

SQL regeneration is available on-demand via gen_config.py --dab-sql for audit/validation purposes.

Tasks

  • Delete bundle/sql/ directory and all contents
  • Update .gitignore to ignore bundle/sql/ (generated on demand)
  • Verify gen_config.py --check still passes (only checks databricks.yml, not SQL)
  • Update any documentation referencing bundle/sql/
## Problem 208 SQL files in `bundle/sql/` are no longer referenced by `databricks.yml` tasks (switched to PythonWheelTask in #223). The files remain in the repo as dead weight. SQL regeneration is available on-demand via `gen_config.py --dab-sql` for audit/validation purposes. ## Tasks - [ ] Delete `bundle/sql/` directory and all contents - [ ] Update `.gitignore` to ignore `bundle/sql/` (generated on demand) - [ ] Verify `gen_config.py --check` still passes (only checks databricks.yml, not SQL) - [ ] Update any documentation referencing bundle/sql/
kert added this to the P28: DAB Modernization — Python wheel tasks, databricks-bundles, schema cleanup milestone 2026-03-25 04:00:41 +00:00
Author
Owner

Starting. Deleting bundle/sql/ and adding to .gitignore.

Starting. Deleting bundle/sql/ and adding to .gitignore.
Author
Owner

Investigation: bundle/sql/ is already in .gitignore (line 14: bundle/sql/). The 208 SQL files were never committed to git — they are local-only generated artifacts.

git ls-files bundle/sql/ | wc -l → 0.

No action needed. The #223 checklist item "delete bundle/sql/" was misleading — the files are local dev artifacts, not repo content.

✅ Already correct.

Investigation: `bundle/sql/` is already in `.gitignore` (line 14: `bundle/sql/`). The 208 SQL files were never committed to git — they are local-only generated artifacts. `git ls-files bundle/sql/ | wc -l` → 0. No action needed. The #223 checklist item "delete bundle/sql/" was misleading — the files are local dev artifacts, not repo content. ✅ Already correct.
kert closed this issue 2026-03-25 04:04:38 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: homelab/stack#228