bobbai00 opened a new pull request, #3824:
URL: https://github.com/apache/texera/pull/3824

   ## Summary
   This PR eliminates the duplication of SQL DDL files between 
`core/scripts/sql/` and `deployment/k8s/texera-helmchart/files/` by replacing 
duplicate files with symlinks.
   
   ## Changes
   - Replaced the duplicate `texera_ddl.sql` in the Helm chart directory with 
symlinks to the core SQL files
   - Created symlinks for all three SQL initialization files:
     - `texera_lakefs.sql` → `../../../../core/scripts/sql/texera_lakefs.sql`
     - `iceberg_postgres_catalog.sql` → 
`../../../../core/scripts/sql/iceberg_postgres_catalog.sql`
     - `texera_ddl.sql` → `../../../../core/scripts/sql/texera_ddl.sql`
   - Updated the Helm chart ConfigMap to properly reference and execute these 
symlinked files in the correct order
   
   ## Benefits
   - **Single source of truth**: All SQL DDL files are maintained only in 
`core/scripts/sql/`
   - **No duplication**: Eliminates maintenance burden of keeping two copies in 
sync
   - **Automatic updates**: Any changes to core SQL files automatically reflect 
in Helm deployments
   - **Clean structure**: Files maintain their original, descriptive names
   
   ## Test plan
   - [ ] Verify symlinks are correctly created and point to the right files
   - [ ] Test Helm chart deployment to ensure databases are initialized 
correctly
   - [ ] Confirm all three databases (LakeFS, Iceberg catalog, Texera) are 
created in the correct order
   
   Fixes #3823
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to