Skip to content

fix: harden warehouse SQL identifier quoting - #7187

Open
rudder-devbox[bot] wants to merge 16 commits into
masterfrom
harness/INT-6791-fix-harden-warehouse-sql-identi-362a1da7
Open

fix: harden warehouse SQL identifier quoting#7187
rudder-devbox[bot] wants to merge 16 commits into
masterfrom
harness/INT-6791-fix-harden-warehouse-sql-identi-362a1da7

test: bound redshift schema ddl checks

40dd7db
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 5, 2026 in 0s

78.92% (-0.91%) compared to ac4a859

View this Pull Request on Codecov

78.92% (-0.91%) compared to ac4a859

Details

Codecov Report

❌ Patch coverage is 81.25000% with 129 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.92%. Comparing base (ac4a859) to head (40dd7db).

Files with missing lines Patch % Lines
warehouse/integrations/redshift/redshift.go 18.04% 109 Missing ⚠️
warehouse/integrations/snowflake/snowflake.go 84.26% 14 Missing ⚠️
...ehouse/integrations/azure-synapse/azure-synapse.go 97.91% 2 Missing ⚠️
warehouse/integrations/mssql/mssql.go 97.87% 2 Missing ⚠️
warehouse/integrations/bigquery/bigquery.go 95.45% 1 Missing ⚠️
warehouse/integrations/clickhouse/clickhouse.go 96.96% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7187      +/-   ##
==========================================
- Coverage   79.82%   78.92%   -0.91%     
==========================================
  Files         600      600              
  Lines       67013    67009       -4     
==========================================
- Hits        53496    52884     -612     
- Misses      10372    11041     +669     
+ Partials     3145     3084      -61     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.