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

[Feature]: Caching of SHOW WAREHOUSES to significantly increase performance #3169

Open
1 task
rd-thomas-uhren opened this issue Nov 4, 2024 · 1 comment
Open
1 task
Labels
feature-request Used to mark issues with provider's missing functionalities

Comments

@rd-thomas-uhren
Copy link

Use Cases or Problem Statement

We observed and confirmed by engineering that the performance of "SHOW WAREHOUSES" and "SHOW WAREHOUSES LIKE 'WAREHOUSE_NAME'" is identical. Since we have a lot of warehouses (~1000) the command ("SHOW WAREHOUSES LIKE 'WAREHOUSE_NAME'") takes approximately 10 seconds to complete; again for each of the 1000 warehouses.

As a result, the build step takes a very long time and is responsible for more than 80% of the total running time.

Engineering proposed to run the SHOW WAREHOUSES once and query the result cache for subsequent calls.

This would greatly improve the running times.

Category

category:resource

Object type(s)

No response

Proposal

Engineering proposed to run the SHOW WAREHOUSES once and query the result cache for subsequent calls.

How much impact is this issue causing?

Medium

Additional Information

No response

Would you like to implement a fix?

  • Yeah, I'll take it 😎
@rd-thomas-uhren rd-thomas-uhren added the feature-request Used to mark issues with provider's missing functionalities label Nov 4, 2024
@sfc-gh-jmichalak
Copy link
Collaborator

Hi @rd-thomas-uhren 👋

This is related to #3118. As announced on our roadmap, we plan to target the performance of running similar/subsequent queries. Caching is one of the potential solutions to improve the performance. But the options currently on the table are: caching, batching requests, and/or not reading the state initially at all. They all have their pros, cons, and challenges.

Unfortunately, we can't improve the performance of a single SHOW WAREHOUSES query for now. However, if you split your deployment into smaller ones, the number of such queries will be reduced, leading to a potential time improvement of the plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Used to mark issues with provider's missing functionalities
Projects
None yet
Development

No branches or pull requests

2 participants