pelias / pelias-ios-sdk

Interact with Mapzen's search & geocoding service
https://mapzen.com/products/search/
Other
9 stars 9 forks source link

Implement Rate Limiting for Auto Complete queries #21

Closed msmollin closed 8 years ago

ecgreb commented 8 years ago

Should we also add logic to cancel

msmollin commented 8 years ago

@ecgreb Cancel is probably a good idea, although we should probably add it for the entire SDK since we can't cancel any enqueued requests. I've made a new issue to address it in a future PR - https://github.com/pelias/pelias-ios-sdk/issues/22

ecgreb commented 8 years ago

Sounds like a plan. Another bit of food for thought: how can we unit test behavior that depends on something inherently hard to test like a timer?