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

Directory Caching trait #24

Merged
merged 8 commits into from
Nov 10, 2023
Merged

Directory Caching trait #24

merged 8 commits into from
Nov 10, 2023

Conversation

nyurik
Copy link
Collaborator

@nyurik nyurik commented Nov 8, 2023

This cache implementation adds a new DirectoryCache trait and a few helpers. Directory and Entry are now public, but neither have any public functions except for find_tile_id which cache should use while locking its internal state if needed.

@nyurik nyurik changed the title [wip] Directory Caching Directory Caching trait Nov 8, 2023
@nyurik nyurik force-pushed the cache branch 3 times, most recently from 4f6a7e3 to 2786f24 Compare November 8, 2023 22:01
@nyurik nyurik marked this pull request as ready for review November 8, 2023 22:03
@nyurik nyurik force-pushed the cache branch 4 times, most recently from 0909c54 to 76c5130 Compare November 10, 2023 08:53
Copy link
Member

@lseelenbinder lseelenbinder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks solid. One suggestion for the HashMap cache.

src/cache.rs Outdated Show resolved Hide resolved
@nyurik nyurik merged commit e7f9444 into stadiamaps:main Nov 10, 2023
1 check passed
@nyurik nyurik deleted the cache branch November 10, 2023 09:25
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

Successfully merging this pull request may close these issues.

None yet

2 participants