Skip to content

Performance: Opt when multiple same URLs queried in parallel, which may cause duplicated decode and store disk operation#3523

Merged
dreampiggy merged 1 commit into
SDWebImage:masterfrom
dreampiggy:opt/same_url_query_extra_decode_store
May 5, 2023
Merged

Performance: Opt when multiple same URLs queried in parallel, which may cause duplicated decode and store disk operation#3523
dreampiggy merged 1 commit into
SDWebImage:masterfrom
dreampiggy:opt/same_url_query_extra_decode_store

Conversation

@dreampiggy

Copy link
Copy Markdown
Contributor

New Pull Request Checklist

  • I have read and understood the CONTRIBUTING guide

  • I have read the Documentation

  • I have searched for a similar pull request in the project and found none

  • I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

  • I have added the required tests to prove the fix/feature I am adding

  • I have updated the documentation (if necessary)

  • I have run the tests and they pass

  • I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: ...

Pull Request Description

This is the quick find and opt for the user reply from #3515

There are other opt ways , need to merge one by one

… cause duplicated decode and store disk operation
@dreampiggy dreampiggy added this to the 5.15.7 milestone May 5, 2023
@dreampiggy dreampiggy added the cache Anything related to Cache System label May 5, 2023
@dreampiggy dreampiggy changed the title Performance: Opt when multiple same URL query in parallels, which may cause duplicated decode and store disk operation Performance: Opt when multiple same URLs queried in parallel, which may cause duplicated decode and store disk operation May 5, 2023
@dreampiggy
dreampiggy merged commit 5481aa2 into SDWebImage:master May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cache Anything related to Cache System performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant