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

Share a single blob URL between all workers #3239

Merged
merged 1 commit into from
Sep 22, 2016
Merged

Share a single blob URL between all workers #3239

merged 1 commit into from
Sep 22, 2016

Conversation

jfirebaugh
Copy link
Contributor

Share a single blob URL between all workers (refs #3153). Turns out this is easy to do, so we might as well, even if the performance benefit is not significant.

Launch Checklist

  • briefly describe the changes in this PR
  • post benchmark scores
  • manually test the debug page

Benchmarks

map-load

master 56e9761: 212 ms
share-blob 9a7af59: 71 ms

style-load

master 56e9761: 102 ms
share-blob 9a7af59: 112 ms

buffer

master 56e9761: 995 ms
share-blob 9a7af59: 991 ms

fps

master 56e9761: 60 fps
share-blob 9a7af59: 60 fps

frame-duration

master 56e9761: 7.3 ms, 0% > 16ms
share-blob 9a7af59: 7.4 ms, 0% > 16ms

query-point

master 56e9761: 0.98 ms
share-blob 9a7af59: 1.12 ms

query-box

master 56e9761: 70.14 ms
share-blob 9a7af59: 86.60 ms

geojson-setdata-small

master 56e9761: 14 ms
share-blob 9a7af59: 14 ms

geojson-setdata-large

master 56e9761: 121 ms
share-blob 9a7af59: 119 ms

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.

2 participants