metricq/metricq-grafana

Make /search?limit configurable

Closed this issue · 2 comments

tilsche commented

While we're at enhancing /search for #29 it's probably a good idea to make the limit configurable, but keep the default of None / 100 for selector and infix requests respectively.

kinnarr commented

Should it be possible for infix to set the limit to None?

tilsche commented

Should it be possible for infix to set the limit to None?

Yes. I suppose you want it to mean "unlimited". But it can be careful to distinguish None from not given at times. Maybe use None for default and false for unlimited in the first place.