Skip to content
This repository was archived by the owner on Oct 1, 2018. It is now read-only.

Speed up android initial assets copy#204

Merged
nikDemyankov merged 2 commits into
nordnet:masterfrom
StuffAndOtherThings:master
Aug 30, 2016
Merged

Speed up android initial assets copy#204
nikDemyankov merged 2 commits into
nordnet:masterfrom
StuffAndOtherThings:master

Conversation

@Mikey1982

Copy link
Copy Markdown
Contributor

Androids AssetManager.list method is known to have performance issues, leading to ASSETS_FOLDER_IN_NOT_YET_INSTALLED error in this case.

Initial assets copy uses now java.util.jar.JarFile to enumerate and extract
assets, wich is much faster.

Mikey1982 added 2 commits August 26, 2016 15:19
Androids AssetManager.list method is known to have performance issues.
Initial assets copy uses java.util.jar.JarFile to enumerate and extract
assets.
@nikDemyankov

Copy link
Copy Markdown
Member

Nice! Thanks for that!

@nikDemyankov

Copy link
Copy Markdown
Member

Can you share some time comparisons on before and after the changes?

@Mikey1982

Copy link
Copy Markdown
Contributor Author

Sure, and first, thanks for you plugin - it saved me a lot of work ;).

In our case (2k+ files in 500+ directories) initial copy took over 70 seconds in emulator and never came back on test devices. With those changes it takes under 5 seconds in emulator and few seconds on test devices.

@nikDemyankov

Copy link
Copy Markdown
Member

Awesome, thanks!

@nikDemyankov nikDemyankov added this to the v1.5.0 milestone Aug 29, 2016
@nikDemyankov nikDemyankov merged commit 56633c5 into nordnet:master Aug 30, 2016
@nikDemyankov

Copy link
Copy Markdown
Member

Released in v1.5.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants