NaN on functions like sum, avg, p90 ... treats numbers as strings? about angle-grinder HOT 3 CLOSED

I think the issue is your pipeline -- after count by status, there is no request time column anymore.

You may want something like avg(request_time) by status or just avg(request_time) (without having count by status first)

from angle-grinder.

If anyone wants to duplicate, here is my nginx log_format:

log_format json_combined escape=json
  '{'
    '"time_local":"$time_local",'
    '"remote_addr":"$remote_addr",'
    '"remote_user":"$remote_user",'
    '"request":"$request",'
    '"status":$status,'
#    '"body_bytes_sent":"$body_bytes_sent",'
    '"request_time":$request_time,'
#    '"upstream_addr":"$upstream_addr",'
#    '"upstream_status":"$upstream_status",'
#    '"upstream_response_time":"$upstream_response_time",'
#    '"upstream_cache_status":"$upstream_cache_status",'
#    '"uid":"$cookie_bbuserid",'
#    '"http_referrer":"$http_referer",'
    '"http_user_agent":"$http_user_agent"'
  '}';

I've commented out nearly everything trying to make a simple test work with agrind. Please note the inconsistency in strings vs ints on numerical fields is just me testing trying to make something work -- in the end, no difference it would seem.

from angle-grinder.

I think the issue is your pipeline -- after count by status, there is no request time column anymore.

You may want something like avg(request_time) by status or just avg(request_time) (without having count by status first)

Dang, I could have sworn I tried exactly that... but nope, just tried and it worked. Or perhaps I tried it with quoted 'strings' instead of numeric values? Maybe that would still break, and I didn't repeat the test without the quotes...

Either way, your suggestion solved it. Thank you

from angle-grinder.