[bisq-network/bisq] Monitor fixes (#3045)

Florian Reimair notifications at github.com
Thu Aug 1 11:49:58 UTC 2019


Finally got the monitor to state that is does not kill some of its metrics after a few hours.

Furthermore, I refactored the P2P Roundtriptime metric so it does not always report all seeds as offline if one or more of them takes too long to respond (the issue showed up just recently, seems that our seeds are under heavier load than they have been).
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/3045

-- Commit Summary --

  * Refactored P2PRTT metric
  * Fixed NullPointer in corner case
  * Fix stopping the schedule because of exception
  * Fixed NullPointerEx of corner case
  * Remove tiny gap in time for missing a message
  * Logging cosmetics
  * Remove debug statement

-- File Changes --

    M monitor/src/main/java/bisq/monitor/Metric.java (16)
    M monitor/src/main/java/bisq/monitor/metric/MarketStats.java (1)
    M monitor/src/main/java/bisq/monitor/metric/P2PRoundTripTime.java (165)
    M monitor/src/main/java/bisq/monitor/metric/P2PSeedNodeSnapshot.java (2)
    M monitor/src/main/java/bisq/monitor/metric/P2PSeedNodeSnapshotBase.java (8)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/3045.patch
https://github.com/bisq-network/bisq/pull/3045.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/3045
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190801/a4cfbfff/attachment.html>


More information about the bisq-github mailing list