I agree that it's a bit strange to have 0 as a fallback value because it can also naturally occur for many metrics. If we want to omit the value null would be probably better as Matthias suggested.
Gyula On Mon, Jul 24, 2023 at 4:02 PM Matthias Pohl <matthias.p...@aiven.io.invalid> wrote: > What was the reason you decided to go for 0 as the fallback value instead > of null? Wouldn't that be a more reasonable value for error cases? > > On Mon, Jul 24, 2023 at 12:51 PM Chesnay Schepler <ches...@apache.org> > wrote: > > > There are a number of cases where the REST API can return infinity or > > NaN for certain double fields. > > > > This is problematic because the JSON spec does not allow such values, > > and tooling working against that spec may run into issues when > > encountering such a value. > > > > Specifically we've seen this become an issue in clients generated from > > the OpenAPI spec. > > > > > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=263425797 > > >
