Skip to content

Commit

Permalink
Reverted the addition of extra function to the ast for #2956.
Browse files Browse the repository at this point in the history
Accidentally added due to ignorance really.
  • Loading branch information
henrikateniro authored and jwilder committed Jun 25, 2015
1 parent 956b716 commit 8bb2fc5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion influxql/ast.go
Expand Up @@ -949,7 +949,7 @@ func (s *SelectStatement) validateAggregates(tr targetRequirement) error {
for _, f := range s.Fields {
if c, ok := f.Expr.(*Call); ok {
switch c.Name {
case "derivative", "non_negative_derivative", "non_integer_derivative":
case "derivative", "non_negative_derivative":
if min, max, got := 1, 2, len(c.Args); got > max || got < min {
return fmt.Errorf("invalid number of arguments for %s, expected at least %d but no more than %d, got %d", c.Name, min, max, got)
}
Expand Down
2 changes: 1 addition & 1 deletion influxql/engine_test.go
Expand Up @@ -208,7 +208,7 @@ func TestProcessDerivative(t *testing.T) {
},
{
name: "float derivatives",
fn: "non_integer_derivative",
fn: "derivative",
interval: "1d",
in: [][]interface{}{
[]interface{}{
Expand Down

0 comments on commit 8bb2fc5

Please sign in to comment.