search

HelixNetwork / pendulum

Pendulum is a distributed messaging protocol that enables globally available tamper proof timestamps :hourglass_flowing_sand:
https://dev.hlx.ai
Other
10 stars 6 forks source link

add trace msgs for balance inconsistency checks #209

Closed dzhelezov closed 4 years ago

dzhelezov commented 4 years ago

Additional trace logging to facilitate debugging for pendulum#208

dzhelezov commented 4 years ago

Agree, trace is currently used for metrics at the downstream log server and as such is not intended to be human readable. Here I used debug msg where I believe an unexpected (and thus relatively rare) event happens, so it's more visible. I might be wrong though on wether it's frequent or not

oracle58 commented 4 years ago

Agree, trace is currently used for metrics at the downstream log server and as such is not intended to be human readable. Here I used debug msg where I believe an unexpected (and thus relatively rare) event happens, so it's more visible. I might be wrong though on wether it's frequent or not

Yep, that it is kind of what it boils down to. We should just consider how fine grained the information we are logging is and base our decision off of that.