Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

Metric adm::get_tiles calls #138

Closed
pjenvey opened this issue Jun 7, 2021 · 2 comments · Fixed by #157
Closed

Metric adm::get_tiles calls #138

pjenvey opened this issue Jun 7, 2021 · 2 comments · Fixed by #157
Assignees
Labels
enhancement New feature or request

Comments

@pjenvey
Copy link
Member

pjenvey commented Jun 7, 2021

Visibility into how frequently we're querying for tiles from ADM would be useful, let's add a simple counter metric for this.

┆Issue is synchronized with this Jira Task

@pjenvey pjenvey added the enhancement New feature or request label Jun 7, 2021
@jrconlin
Copy link
Member

jrconlin commented Jun 8, 2021

There is a metric call on handlers/get_tiles(), which should suffice. Do you want one that is explicitly for adm?

@pjenvey
Copy link
Member Author

pjenvey commented Jun 11, 2021

There is a metric call on handlers/get_tiles(), which should suffice. Do you want one that is explicitly for adm?

Yea, since they're isn't a 1:1 mapping of handler::get_tiles calls to the call to adm (adm::get_tiles)

@pjenvey pjenvey changed the title Metric get_tiles calls Metric adm::get_tiles calls Jun 11, 2021
@pjenvey pjenvey self-assigned this Jun 16, 2021
pjenvey added a commit that referenced this issue Jun 16, 2021
pjenvey added a commit that referenced this issue Jun 16, 2021
and some naming cleanup

Closes #138
pjenvey added a commit that referenced this issue Jun 17, 2021
and some naming cleanup

Closes #138
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants