Skip to content

Switched to AsyncKeyedLock library for better performance#106

Open
MarkCiliaVincenti wants to merge 4 commits into
RWS:developfrom
MarkCiliaVincenti:AsyncKeyedLock
Open

Switched to AsyncKeyedLock library for better performance#106
MarkCiliaVincenti wants to merge 4 commits into
RWS:developfrom
MarkCiliaVincenti:AsyncKeyedLock

Conversation

@MarkCiliaVincenti

Copy link
Copy Markdown

No description provided.

@MarkCiliaVincenti

Copy link
Copy Markdown
Author

@majiccode please let me know

@avmgan

avmgan commented Feb 12, 2024

Copy link
Copy Markdown
Collaborator

current develop branch merged DXA 2.3 release related changes, it's conflicting your changes.
I think better we can keep AsynKeylockCacheProvider separate class, people can choose which caching machanism they want rather than overriding the exisitng options, if you raise a new PR by keeping the new class, I can review and merge it. if not I can plan to this change sometime later.

@majiccode

Copy link
Copy Markdown
Contributor

@MarkCiliaVincenti looks good to me :)

@MarkCiliaVincenti

Copy link
Copy Markdown
Author

@MarkCiliaVincenti looks good to me :)

Check now pls.

@MarkCiliaVincenti

Copy link
Copy Markdown
Author

@majiccode

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.

3 participants