search

prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
15.85k stars 5.32k forks source link

NaN in coordinator UI on load #23186

Closed elharo closed 4 weeks ago

elharo commented 1 month ago

When the coordinator UI first loads it shows "NaN" for a number of values until this is resolved with initial values a few seconds in.

My guess is that it starts with default values of zero for a number of variables, divides zero by zero, gets NaN, and loads that straight into the UI without checking. It should probably instead check for NaN and display an empty string or a no data message instead. Or maybe it should check for zero divisor before it even does the division.

Screenshot 2024-07-11 at 7 50 33 AM
elharo commented 1 month ago

suspect the issue is in the precisionRound function in utils.js which does not handle NaN

Probably this should return "Unknown" or "Not available" on NaN.

anilsomisetty commented 1 month ago

Hey @elharo, Could you please assign this issue to me ?