Thanks Chettam for fixing we saw this problem in prod. +1
On Fri, Mar 8, 2019 at 8:22 PM Rodric Rabbah <[email protected]> wrote: > Chetan opened this PR > https://github.com/apache/incubator-openwhisk/pull/4327 which fixes an > issue with using historgrams for some metrics where gauges should be used > instead. There was a brief exchange on slack [1, 2] in favor of this > change. > > The PR has two commits, the first I think is fine and non-breaking. The > second change as Chetan notes is "sort of a breaking change" but I agree > which him in that the existing histograms do not look very useful in some > cases. > > If you're interested in this PR please have a look and comment, otherwise > will apply lazy consensus to merge it. > > -r > > [1] > > https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1552009389150600?thread_ts=1551982869.141200&cid=C3TPCAQG1 > > [2] > > https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1552014636150800?thread_ts=1551982869.141200&cid=C3TPCAQG1 > -- Carlos Santana <[email protected]>
