-
Notifications
You must be signed in to change notification settings - Fork 78
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
mapKnitterLayer #13
mapKnitterLayer #13
Conversation
@jywarren ...kindly see this . |
this looks amazing! Let's not worry too much about the icons -- we can
change them in follow-up as we get input from people.
Are you concerned about the load this would put on MapKnitter's API?
…On Thu, Jun 14, 2018 at 1:36 AM Sagarpreet Chadha ***@***.***> wrote:
@jywarren <https://github.com/jywarren> ...kindly see this .
We can change the marker - icon later . What do you think ?
[image: screen shot 2018-06-14 at 11 58 54 am]
<https://user-images.githubusercontent.com/14952645/41395368-0ae93b2e-6fcb-11e8-99e7-64323349b1f8.png>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#13 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJ8u35GhIfuCRwVdfjBOqCDxG2FiBks5t8gRngaJpZM4UnYVg>
.
|
I guess an average user in the worst case will make around 5 hits per second to the server and if 50 users are using this layer simultaneously then it will add to 250 hits per second . |
I think that's probably right. I think it should be OK, but we'll keep an
eye out and we can do some caching to respond to this if need be. All good
then, thanks!
…On Sat, Jun 16, 2018 at 3:04 AM Sagarpreet Chadha ***@***.***> wrote:
I guess an average user in the worst case will make around 5 hits per
second to the server and if 50 users are using this layer simultaneously
then it will add to 250 hits per second .
I think this should not put much load on the server , but i am not sure if
this is the correct way to estimate the load . What do you think ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#13 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJ-Mic1ldSDsaQzlvsKwGR0dooOHjks5t9K3AgaJpZM4UnYVg>
.
|
solves #10 .