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

Implement caching for dynamic views #332

Open
deepu105 opened this issue Jul 5, 2023 · 2 comments
Open

Implement caching for dynamic views #332

deepu105 opened this issue Jul 5, 2023 · 2 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@deepu105
Copy link
Contributor

deepu105 commented Jul 5, 2023

Dynamic views are not cached and hence on load, there is a small delay. It would be nice to iterate through and cache these in a BTreemap and update on subsequent fetch.

@deepu105 deepu105 added good first issue Good for newcomers help wanted Extra attention is needed labels Jul 5, 2023
@Narayanbhat166
Copy link

Hi @deepu105. I'm interested in working on this. Can you please provide more information as to what are the components that I need to look into?

@deepu105
Copy link
Contributor Author

deepu105 commented Aug 23, 2023

@Narayanbhat166 great! I recently refactored the codebase so that its easier to work on individual components. You can start from https://github.com/kdash-rs/kdash/blob/main/src/network/mod.rs#L322 and https://github.com/kdash-rs/kdash/blob/main/src/app/dynamic.rs#L104

currently dynamic resources are saved on to the same field and hence overwrites when you switch to another leading to fresh fetch all the ttim. Ideally this line should become a BTreemap of StatefulTable so that during lookup of dynamic kinds you can run a fetch for all discovered types and store them by kind on to the Btreemap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants