Skip to content

Conversation

achettyiitr
Copy link
Member

@achettyiitr achettyiitr commented Sep 30, 2025

Description

Linear Ticket

  • Resolves WAR-1186

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@achettyiitr achettyiitr changed the title chore: table level schemas chore: improve table level schemas Sep 30, 2025
@achettyiitr achettyiitr force-pushed the chore.table-level-schemas branch from 579618e to 1ec62af Compare September 30, 2025 05:41
@@ -0,0 +1,4 @@
-- Add index on wh_schemas table to speed up queries on destination_id, namespace, and table_name columns (e.g. for schema snapshot query)
CREATE INDEX CONCURRENTLY IF NOT EXISTS wh_schemas_dest_namespace_table_latest_idx
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we call the index "latest"? If we add another index in the future, it might be confusing

Copy link

codecov bot commented Sep 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.60%. Comparing base (9017582) to head (1ec62af).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6398      +/-   ##
==========================================
- Coverage   77.64%   77.60%   -0.04%     
==========================================
  Files         523      523              
  Lines       70313    70313              
==========================================
- Hits        54593    54569      -24     
- Misses      12879    12913      +34     
+ Partials     2841     2831      -10     

☔ View full report in Codecov by Sentry.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants