Skip to content

Commit

Permalink
Allow the CQ to be missing as well
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewhare committed Sep 26, 2017
1 parent 938bd1d commit e135cde
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions services/meta/client_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -809,9 +809,9 @@ func TestMetaClient_ContinuousQueries(t *testing.T) {
t.Fatal(err)
}

// Dropping a nonexistent CQ should return an error.
if err := c.DropContinuousQuery("db0", "not-a-cq"); err == nil {
t.Fatal("expected an error, got nil")
// Dropping a nonexistent CQ should not return an error.
if err := c.DropContinuousQuery("db0", "not-a-cq"); err != nil {
t.Fatal(err)
}
}

Expand Down
2 changes: 1 addition & 1 deletion services/meta/data.go
Original file line number Diff line number Diff line change
Expand Up @@ -448,7 +448,7 @@ func (data *Data) DropContinuousQuery(database, name string) error {
return nil
}
}
return ErrContinuousQueryNotFound
return nil
}

// validateURL returns an error if the URL does not have a port or uses a scheme other than UDP or HTTP.
Expand Down

0 comments on commit e135cde

Please sign in to comment.