[bisq-network/bisq] periodical UpdateDataReq for Seednodes (#3202)

Christoph Sturm notifications at github.com
Sat Sep 7 12:23:40 UTC 2019


christophsturm commented on this pull request.



> +            });
+        }
+
+        // Log different data types
+        StringBuilder sb = new StringBuilder();
+        sb.append("\n#################################################################\n");
+        sb.append("Connected to node: " + peersNodeAddress.getFullAddress() + "\n");
+        final int items = dataSet.size() +
+                (persistableNetworkPayloadSet != null ? persistableNetworkPayloadSet.size() : 0);
+        sb.append("Received ").append(items).append(" instances\n");
+        payloadByClassName.entrySet().stream().forEach(e -> sb.append(e.getKey())
+                .append(": ")
+                .append(e.getValue().size())
+                .append("\n"));
+        sb.append("#################################################################");
+        log.info(sb.toString());

info is probably enabled most of the time. still this whole method should be in a `if (log.isInfoEnabled)` block

-- 
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/3202#pullrequestreview-285169732
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190907/2c7e2bb2/attachment.html>


More information about the bisq-github mailing list