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.

Sign up for GitHub

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

Jump to bottom

Use error codes rather than checking message #42

Open
yhahn opened this issue Apr 18, 2012 · 0 comments
Open

Use error codes rather than checking message #42

yhahn opened this issue Apr 18, 2012 · 0 comments

Comments

@yhahn
Copy link
Member

yhahn commented Apr 18, 2012

Currently in tilelive and tilelive API-implementing modules an error with message Tileset does not exist (and relatedly, Tile does not exist, Grid does not exist) often get special treatment.

Look into adding a code attribute or so that makes checking for these errors less freaky.
robpvn added a commit to Norkart/tilelive.js that referenced this issue Aug 7, 2015
@robpvn
Make implicit API expectation implicit
6bf7e6c 
I was bitten by issue mapbox#42 when trying to implement a tilelive TileSource that used vector tiles, which by their nature are sparse. (My backend would faithfully emit error messages for missing tiles, but with a different wording which med tilelive crash out rather than just skip the tile.) Until the actual issue is resolved (which would mean rewriting all tilelive implementations, I guess) it should be made an explicit part or the API doc.
@robpvn robpvn mentioned this issue Aug 7, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yhahn