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

Add request throttling for autocomplete #28

Open
ecgreb opened this issue Jan 28, 2016 · 3 comments
Open

Add request throttling for autocomplete #28

ecgreb opened this issue Jan 28, 2016 · 3 comments

Comments

@ecgreb
Copy link
Collaborator

ecgreb commented Jan 28, 2016

No description provided.

@missinglink
Copy link
Member

kingofirony added a commit that referenced this issue Feb 24, 2016
@sarahsnow1
Copy link
Member

probably related...i just got this crash:

06-13 10:50:43.247 18032-18032/com.mapzen.erasermap E/PeliasSearchView: Unable to fetch autocomplete results
                                                                        retrofit.RetrofitError: 429 Too Many Requests
                                                                            at retrofit.RestAdapter$RestHandler.invokeRequest(RestAdapter.java:388)
                                                                            at retrofit.RestAdapter$RestHandler.access$100(RestAdapter.java:220)
                                                                            at retrofit.RestAdapter$RestHandler$2.obtainResponse(RestAdapter.java:278)
                                                                            at retrofit.CallbackRunnable.run(CallbackRunnable.java:42)
                                                                            at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1113)
                                                                            at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:588)
                                                                            at retrofit.Platform$Android$2$1.run(Platform.java:142)
                                                                            at java.lang.Thread.run(Thread.java:818)

@ecgreb ecgreb added the next label Jan 17, 2017
@ecgreb
Copy link
Collaborator Author

ecgreb commented Jan 17, 2017

I am getting this crash even more now when testing the Places API for Android since it is hammering the autocomplete endpoint.

Now that we have upgraded to Retrofit2/OkHttp3 we should address this sooner than later.

@ecgreb ecgreb removed the next label Feb 1, 2017
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

No branches or pull requests

3 participants