Skip to content
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

Replace service_type = 'COMPUTE' filters with service_type = 'WAREHOUSE_METERING' due to Snowflake backend change #150

Open
stumelius opened this issue Apr 2, 2024 · 5 comments

Comments

@stumelius
Copy link
Contributor

The COMPUTE service type no longer exists in the snowflake.organization_usage.rate_sheet_daily table. It has been replaced with WAREHOUSE_METERING.

@stumelius
Copy link
Contributor Author

I can do a PR for this tomorrow :)

@ian-whitestone
Copy link
Contributor

@stumelius as an FYI, there was a related discussion here: #149

Right now the latest version of the package is setup to handle this and ensure we are able to continue to report on spend.

With that said, I would like to update the models at some point now that everyone should have these new service types populated. If you're open to doing a PR, that would be greatly appreciated. We'd need to do the following:

  1. Update stg_rate_sheet_daily to undo the manually handling
  2. Update cost_per_query
  3. Update hourly_spend

@stumelius
Copy link
Contributor Author

@ian-whitestone Oh, I didn't notice the service_type transformation in stg_rate_sheet_daily. So, as a quick fix I'll just update the package to the latest version and that should do the trick.

Here's a PR for the long term fix #151

@ian-whitestone
Copy link
Contributor

Thanks so much @stumelius! Will aim to review this weekend when things are quieter :)

@greg-finley
Copy link

Hello, seems that this issue can be closed?

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

No branches or pull requests

3 participants