Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

fix(catalog): Mini catalog is taking a long time to load #1353

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

igarashitm
Copy link
Contributor

@igarashitm igarashitm commented Mar 2, 2023

Fixes: #1269

@igarashitm
Copy link
Contributor Author

With this change, only the first attempt to open a mini catalog fetch it from backend, but the following just uses the cached catalog in the stepCatalogStore, which is cleared if DSL type is changed.

@igarashitm igarashitm marked this pull request as ready for review March 3, 2023 03:55
@igarashitm igarashitm requested a review from a team March 3, 2023 03:55
@sonarcloud
Copy link

sonarcloud bot commented Mar 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@igarashitm igarashitm enabled auto-merge (rebase) March 3, 2023 03:56
@igarashitm igarashitm merged commit cd04e57 into KaotoIO:main Mar 3, 2023
@igarashitm
Copy link
Contributor Author

Ugh I didn't meant to merge until it gets reviewed 😅

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

Successfully merging this pull request may close these issues.

Catalog: Mini catalog is taking a long time to load
1 participant