Nick Johnson

When looking for indexes to satisfy a query, BDBDatastore checks all indexes. For more efficiency, indexes could be indexed by property type and sorted list of fields.

When attempting to enqueue upgrade tasks etc based on a user visiting the admin console, gracefully degrade and do nothing if there's a transient error.

Go expvar parsing does not work on metrics containing '/'

1

https://github.com/DataDog/dd-agent/blob/master/checks.d/go_expvar.py#L132 `parse_expvar_data` splits the provided metric path on '/' and passes it to `deep_get`, which assumes each element in the metrics dict it's passed corresponds to a single path element,...

Variables don't do typechecking

5

While other blocks check types (eg, if, comparison), any block will accept a variable. For instance, you can't do "if(0)", but you can do "if(x)" where x is an int.

Consider removing no-op grove blocks

6

Some of the grove blocks, such as the ultrasonic rangefinder, clearly add a lot of value, because they provide an interface to library functionality. A lot of them, though, such...