Skip to content

Stddev function accepts strings #7436

Closed
@Tomcat-Engineering

Description

@Tomcat-Engineering

This line in the source code specifically allows the stddev (standard deviation) function to accept time series of datatype "string".

Obviously there is no sensible way to take the standard deviation of a load of strings, and the implementation always returns an empty string.

This is inconsistent with all the other comparable functions (such as mean), which throw an error if you feed them strings.

Was there a reason for this special case?

Activity

jsternberg

jsternberg commented on Oct 10, 2016

@jsternberg
Contributor

Looks like the piece of code that allows stddev() to operate on strings has been there since the beginning and I can't find a commit that explains why from a quick look.

@benbjohnson do you have any idea?

benbjohnson

benbjohnson commented on Oct 10, 2016

@benbjohnson
Contributor

@jsternberg I can't think of any reason to use a string. We can remove that.

self-assigned this
on Oct 10, 2016
added this to the 1.1.0 milestone on Dec 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions

    Stddev function accepts strings · Issue #7436 · influxdata/influxdb