Closed harudark closed 2 years ago
On second thought - I'm wondering if there's a mistake in the docs.
Drop datapoints where the corresponding value in the second series set is non-zero. (See Series Operations for what corresponding means).
The behaviour currently seems to be that the points are kept (rather than dropped) where the corresponding value in the second series set is non-zero. 🤔 That doesn't change the feedback about this PR, but would be a separate issue.
Edit: Raised #2496
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Description
In production, Bosun got panic with the stacktrace below (I removed the real alert name and expression):
union
is called insidedropbool
and it's possibleunion.A
orunion.B
hasNaN()
value, which is aexpr.Number
, notexpr.Series
.union.A
cannot do type assertion, we ignore this.union.A
can butunion.B
cannot,dropbool
keeps everything inunion.A
.Type of change
How has this been tested?
See
TestDropBool
added in cmd/bosun/expr/funcs_test.goChecklist: