-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[COST-4907] Add temp table to track aws disk capacities #5262
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5262 +/- ##
=======================================
- Coverage 94.1% 94.1% -0.0%
=======================================
Files 375 375
Lines 31422 31429 +7
Branches 4627 4627
=======================================
+ Hits 29564 29570 +6
- Misses 1187 1188 +1
Partials 671 671 |
@@ -162,6 +162,19 @@ CREATE TABLE IF NOT EXISTS hive.{{schema | sqlsafe}}.reporting_ocpawscostlineite | |||
) WITH(format = 'PARQUET', partitioned_by=ARRAY['aws_source', 'ocp_source', 'year', 'month', 'day']) | |||
; | |||
|
|||
{% if unattributed_storage %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to disable table creation? We didn't for Azure, I don't think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put it behind unleash for now so that there is flexibility to add another column if needed later on in the process without having to run a trino migration. If it is a blocker I can remove the if though.
Jira Ticket
COST-4907
Description
This change will add a temporary table for tracking aws disk capacities.
Testing
Example return:
Release Notes