search

mit-dci / opencbdc-tx

A transaction processor for a hypothetical, general-purpose, central bank digital currency
Other
896 stars 198 forks source link

Use logging::log and remove use of std::cerr in raft::node #181

Closed metalicjames closed 1 year ago

metalicjames commented 2 years ago

Fixes #180.

This PR removes the use of std::cerr in raft::node. It also replaces usage of the NuRaft console logger, instead using our own logger class.

HalosGhost commented 2 years ago

@mszulcz-mitre would love a review from you on this to make sure the issue is solved on your end.

mszulcz-mitre commented 1 year ago

@HalosGhost I'd be happy to review. It looks good at first glance, but I'll build and test to make sure.

mszulcz-mitre commented 1 year ago

@HalosGhost I wasn't able to test exhaustively, but the tests I did run looked good.